Uploaded image for project: 'Couchbase Go SDK'
  1. Couchbase Go SDK
  2. GOCBC-1075

buildAuthHandler returns incorrect error object

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • core-9.1.4
    • None
    • None
    • 1

    Description

      See https://github.com/couchbase/gocbcore/blob/341fc70dc8ba195416b4dc8f6d8599e391de686b/agent.go#L449-L466 if there is a callErr then we return the err from getKvAuthCreds (which will be nil). This could cause the consumer of the function to wait on either of the returned channels which will be nil and could lead to waiting on the channels forever.

      Attachments

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

        Activity

          Build couchbase-server-7.0.0-4814 contains gocbcore commit 609458a with commit message:
          GOCBC-1075: Return the correct error from buildAuthHandler

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-4814 contains gocbcore commit 609458a with commit message: GOCBC-1075 : Return the correct error from buildAuthHandler

          Build sync_gateway-3.0.0-225 contains gocbcore commit 609458a with commit message:
          GOCBC-1075: Return the correct error from buildAuthHandler

          build-team Couchbase Build Team added a comment - Build sync_gateway-3.0.0-225 contains gocbcore commit 609458a with commit message: GOCBC-1075 : Return the correct error from buildAuthHandler

          Build couchbase-server-7.1.0-1767 contains gocbcore commit 609458a with commit message:
          GOCBC-1075: Return the correct error from buildAuthHandler

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.1.0-1767 contains gocbcore commit 609458a with commit message: GOCBC-1075 : Return the correct error from buildAuthHandler

          People

            charles.dixon Charles Dixon
            charles.dixon Charles Dixon
            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