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

            There are no comments yet on this issue.

            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