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

SASL password with space not being handled properly

    Details

    • Type: Bug
    • Status: Resolved
    • 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

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

          Hide
          trond Trond Norbye added a comment -

          Why don't we just deny space for now.. It would be way better if we didn't share filesystem layout between modules, and we've got more important tasks to fix in memcached. THis conflicts with all of the other refactoring that is going on in memcached right now and will lead to a pain to move forward.

          Show
          trond Trond Norbye added a comment - Why don't we just deny space for now.. It would be way better if we didn't share filesystem layout between modules, and we've got more important tasks to fix in memcached. THis conflicts with all of the other refactoring that is going on in memcached right now and will lead to a pain to move forward.
          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)

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Gerrit Reviews

                There is 1 open Gerrit change