Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-6823

SASL password with space not being handled properly

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 1.8.1, 2.0, 2.2.0
    • Fix Version/s: bug-backlog
    • Component/s: couchbase-bucket
    • Security Level: Public
    • Labels:
      None
    • Triage:
      Untriaged
    • Link to Log File, atop/blg, CBCollectInfo, Core dump:
      asdasd
    • Is this a Regression?:
      Yes

      Description

      I'm getting a SASL auth error using 'isasl' and libcouchbase.
      The bucket is a memcached bucket with a password including a space in it "a password". When the password is changed to "a_password" it works fine.

      Attached is the JSON bucket output (pools/defaults/buckets/memcached_bucket)

      mnunberg@csure:/sources/libcouchbase$ /opt/couchbase/bin/couchbase-cli bucket-edit --cluster=10.3.121.209 --bucket-password="a password" -u Administrator -p 123456 --bucket="memcached_bucket"
      SUCCESS: bucket-edit
      mnunberg@csure:/sources/libcouchbase$ echo 'hi' | ./tools/cbc create -h 10.3.121.209 -u Administrator -P 123456 a_key -b memcached_bucketERROR: Authentication error
      "SASL authentication failed"
      mnunberg@csure:/sources/libcouchbase$ /opt/couchbase/bin/couchbase-cli bucket-edit --cluster=10.3.121.209 --bucket-password="a_password" -u Administrator -p 123456 --bucket="memcached_bucket"
      SUCCESS: bucket-edit
      mnunberg@csure:/sources/libcouchbase$ echo 'hi' | ./tools/cbc create -h 10.3.121.209 -u Administrator -P 123456 a_key -b memcached_bucketStored "a_key" CAS:100000000000000
      mnunberg@csure:/sources/libcouchbase$

        Issue Links

        # Subject Project Status CR V
        For Gerrit Dashboard: &For+MB-6823=message:MB-6823

          Activity

          Hide
          maria Maria McDuff (Inactive) added a comment -

          Alk, is the plan in placed now for this issue?

          Show
          maria Maria McDuff (Inactive) added a comment - Alk, is the plan in placed now for this issue?
          Hide
          alkondratenko Aleksey Kondratenko (Inactive) added a comment -

          That has nothing to do with ns-server.

          AFAIR it's limitation of isasl.pw format and I recall that Trond had some plans around there. But there's also that in-flight request of hashing bucket passwords.

          IMHO that's not 3.0 thing.

          Show
          alkondratenko Aleksey Kondratenko (Inactive) added a comment - That has nothing to do with ns-server. AFAIR it's limitation of isasl.pw format and I recall that Trond had some plans around there. But there's also that in-flight request of hashing bucket passwords. IMHO that's not 3.0 thing.
          Hide
          chiyoung Chiyoung Seo added a comment -

          Moving this ticket to post 3.0 given the 3.0 release schedule and its scope.

          Show
          chiyoung Chiyoung Seo added a comment - Moving this ticket to post 3.0 given the 3.0 release schedule and its scope.
          Hide
          trond Trond Norbye added a comment -

          We're not going prioritize fixing this (until we replace the backend with a real SASL impl)

          Show
          trond Trond Norbye added a comment - We're not going prioritize fixing this (until we replace the backend with a real SASL impl)
          Hide
          raju Raju Suravarjjala added a comment -

          Bulk closing all invalid bugs that are duplicate, user error, invalid. Please feel free to reopen them if you feel otherwise

          Show
          raju Raju Suravarjjala added a comment - Bulk closing all invalid bugs that are duplicate, user error, invalid. Please feel free to reopen them if you feel otherwise

            People

            • Assignee:
              trond Trond Norbye
              Reporter:
              mnunberg Mark Nunberg
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Gerrit Reviews

                There is 1 open Gerrit change