Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-1131

Regression causes HttpStreamingProvider to auth invalid password

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Blocker
    • Resolution: Fixed
    • 2.2.8
    • 2.3.0
    • None
    • None

    Description

      While running through integration tests for 2.3.0, found a failing test that can be traced back to https://issues.couchbase.com/browse/NCBC-1098

      Github: https://github.com/couchbase/couchbase-net-client/commit/64d48e9dc9ed8ac6a1637413d6a949cfa992bba4

      The failing test: https://github.com/couchbase/couchbase-net-client/blob/master/Src/Couchbase.Tests/ClusterTests.cs#L45

      The net effect here is that the client will try to auth the bucket using CCCP, when it fails it will then try to auth using HttpStreamingProvider in which case it will succeed even though the password is invalid for the bucket.

      Note that if a K/V operation is attempted, it will fail because the SASL authentication will fail on the socket:

      at Couchbase.IO.Services.MultiplexingIOService.Authenticate(IConnection connection) in C:\Users\jmorris\repos\couchbase-net-client\Src\Couchbase\IO\Services\MultiplexingIOService.cs:line 332
      at Couchbase.IO.Services.MultiplexingIOService.Execute[T](IOperation`1 operation) in C:\Users\jmorris\repos\couchbase-net-client\Src\Couchbase\IO\Services\MultiplexingIOService.cs:line 98

      Attachments

        Issue Links

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

          Activity

            jmorris Jeff Morris created issue -
            jmorris Jeff Morris made changes -
            Field Original Value New Value
            Status New [ 10003 ] Open [ 1 ]
            jmorris Jeff Morris made changes -
            Link This issue is triggered by NCBC-1098 [ NCBC-1098 ]
            jmorris Jeff Morris made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            jmorris Jeff Morris made changes -
            Resolution Fixed [ 1 ]
            Status In Progress [ 3 ] Resolved [ 5 ]
            ingenthr Matt Ingenthron made changes -
            Workflow Couchbase SDK Workflow [ 60866 ] Couchbase SDK Workflow with Review [ 246455 ]

            People

              jmorris Jeff Morris
              jmorris Jeff Morris
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty