Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-6407

node-mongodb-native - PR #4258: fix: use conversationId returned by the server instead of hardcoded integer in SASL implementation for MONGODB-AWS.

    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      kggau has created PR #4258: fix: use conversationId returned by the server instead of hardcoded integer in SASL implementation for MONGODB-AWS. in node-mongodb-native

      ### Description
      `MONGODB-AWS` authentication mechanism should follow SASL [spec](https://github.com/mongodb/specifications/blob/master/source/auth/auth.md#sasl-mechanisms). Current implementation ignores the `conversationId` sent by the server and uses a hardcoded integer for all `saslContinue` messages across all conversations.
      
      #### What is changing?
      Changing the hardcoded integer for `conversationId` to utilize the one that gets returned in the `saslStart` response.
      
      ##### Is there new documentation needed for these changes?
      No. This change brings the implementation to spec.
      
      #### What is the motivation for this change?
      
      This is a bug. MONGODB-AWS authentication failed on database emulating MongoDB.
      
      ### Release Highlight
      
      Fixes SASL implementation for MONGODB-AWS to use conversationId returned by the server instead of hardcoded integer.
      

            Assignee:
            Unassigned Unassigned
            Reporter:
            tom.selander@mongodb.com Tom Selander
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated: