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

cbcollect_info fails to collect diags with special char in password

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.1.0
    • Component/s: tools
    • Security Level: Public
    • Labels:
      None
    • Sprint:
      PCI Team - Sprint 5

      Description

      cbcollect_info fails to gather diags if @ sign is in password:

      couchbase diags
      curl -sS "http://Administrator:test@temp@127.0.0.1:8091/diag?noLogs=1"
      ==============================================================================
      curl: (6) Couldn't resolve host 'temp@127.0.0.1'

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

        Activity

        Show
        steve Steve Yen added a comment - See http://cbugg.hq.couchbase.com/bug/bug-164
        Hide
        alkondratenko Aleksey Kondratenko (Inactive) added a comment -

        Recently fixed

        Show
        alkondratenko Aleksey Kondratenko (Inactive) added a comment - Recently fixed
        Hide
        Aliaksey Artamonau Aliaksey Artamonau added a comment -

        We haven't merge the fix for cbcollect_info itself because we wanted to make it more proper.

        Show
        Aliaksey Artamonau Aliaksey Artamonau added a comment - We haven't merge the fix for cbcollect_info itself because we wanted to make it more proper.
        Hide
        alkondratenko Aleksey Kondratenko (Inactive) added a comment -

        IMHO that's bad enough to actually aim for 2.0.2

        Show
        alkondratenko Aleksey Kondratenko (Inactive) added a comment - IMHO that's bad enough to actually aim for 2.0.2
        Show
        alkondratenko Aleksey Kondratenko (Inactive) added a comment - http://review.couchbase.org/25428
        Hide
        maria Maria McDuff (Inactive) added a comment -

        pls verify in current 2.0.2 build. thanks.

        Show
        maria Maria McDuff (Inactive) added a comment - pls verify in current 2.0.2 build. thanks.
        Hide
        thuan Thuan Nguyen added a comment -

        Tested on build 2.0.2-760.
        Setup couchbase server with password=pass@word
        Got error:
        Could not resolve host

        Thuans-MacBook-Pro:testrunner thuan$ curl -sS "http://Administrator:pass@word@10.3.2.49:8091/diag?noLogs=1"
        curl: (6) Could not resolve host: word@10.3.2.49; nodename nor servname provided, or not known

        Show
        thuan Thuan Nguyen added a comment - Tested on build 2.0.2-760. Setup couchbase server with password=pass@word Got error: Could not resolve host Thuans-MacBook-Pro:testrunner thuan$ curl -sS "http://Administrator:pass@word@10.3.2.49:8091/diag?noLogs=1" curl: (6) Could not resolve host: word@10.3.2.49; nodename nor servname provided, or not known
        Hide
        alkondratenko Aleksey Kondratenko (Inactive) added a comment -

        you're supposed to check with cbcollect_info not using old invocation of curl

        Show
        alkondratenko Aleksey Kondratenko (Inactive) added a comment - you're supposed to check with cbcollect_info not using old invocation of curl
        Hide
        thuan Thuan Nguyen added a comment -

        Test on build 2.0.2-760.
        Set password to pass@word, collect_info command works as expected

        memcached stats prev-vbucket (cbstats -a 127.0.0.1:11210 prev-vbucket -b _admin -p _admin) - OK
        memcached stats vbucket (cbstats -a 127.0.0.1:11210 vbucket -b _admin -p _admin) - OK
        memcached stats vbucket details (cbstats -a 127.0.0.1:11210 vbucket-details -b _admin -p _admin) - OK
        memcached stats warmup (cbstats -a 127.0.0.1:11210 warmup -b _admin -p _admin) - OK
        couchbase diags (['curl', '-sS', '-u', 'Administrator:pass@word', 'http://127.0.0.1:8091/diag?noLogs=1']) - OK
        couchbase design docs - bucket:default (couch_dbdump "/opt/couchbase/var/lib/couchbase/data/default/master.couch.1") - OK
        [root@cen-1913 ~]#

        Show
        thuan Thuan Nguyen added a comment - Test on build 2.0.2-760. Set password to pass@word, collect_info command works as expected memcached stats prev-vbucket (cbstats -a 127.0.0.1:11210 prev-vbucket -b _admin -p _admin) - OK memcached stats vbucket (cbstats -a 127.0.0.1:11210 vbucket -b _admin -p _admin) - OK memcached stats vbucket details (cbstats -a 127.0.0.1:11210 vbucket-details -b _admin -p _admin) - OK memcached stats warmup (cbstats -a 127.0.0.1:11210 warmup -b _admin -p _admin) - OK couchbase diags ( ['curl', '-sS', '-u', 'Administrator:pass@word', 'http://127.0.0.1:8091/diag?noLogs=1'] ) - OK couchbase design docs - bucket:default (couch_dbdump "/opt/couchbase/var/lib/couchbase/data/default/master.couch.1") - OK [root@cen-1913 ~] #

          People

          • Assignee:
            thuan Thuan Nguyen
            Reporter:
            james.mauss James Mauss
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Agile

                Gerrit Reviews

                There are no open Gerrit changes