Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-5747 We do not seem to support loading MONGODB-AWS credentials from .aws/credentials
  3. NODE-5944

6.x: Make AWS token optional

XMLWordPrintableJSON

    • Type: Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 6.4.0
    • Affects Version/s: None
    • Component/s: None
    • 0
    • Not Needed
    • v5.x
    • Not Needed
    • 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?

      Use Case

      As a... AWS user
      I want... connect with key and secret and without token

      According to spec:

      Users MAY have obtained temporary credentials through an AssumeRole request. If so, then in addition to a username and password, users MAY also provide an AWS_SESSION_TOKEN as a mechanism_property.

      The AWS_SESSION_TOKEN is optional, so we should not check for it when making temp credentials: https://github.com/mongodb/node-mongodb-native/blob/main/src/cmap/auth/mongodb_aws.ts

      Acceptance Criteria

      Implementation Requirements

      • Remove check that creds.Token is present in makeTempCredentials()

      Testing Requirements

      • Verify that there is no new MongoMissingCredentialsError('Could not obtain temporary MONGODB-AWS credentials') exception in case of a missing token

            Assignee:
            alena.khineika@mongodb.com Alena Khineika
            Reporter:
            alena.khineika@mongodb.com Alena Khineika
            Durran Jordan
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: