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

Investigate the need for aws4 [non-breaking]

    • 3
    • Not Needed
    • 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

      We should investigate the need for https://www.npmjs.com/package/aws4 (aws4) as an optional peer dependency of the driver. When looking into the depths of the package it appears to have a small amount of functionality that we could inline in the driver as part of the AWS auth protocol logic. 

      User Impact

      • This could remove the need for users to have to install aws4 in order to get AWS auth working.
      • It could be done in a feature version

      Dependencies

      • aws4

      Unknowns

      • Is it really a small bit of logic?
      • Any concerns about maintainability?
        • Reporter note: I don't think any other drivers have a dependency for this, the logic here is in our common driver specification, it would need to be changed there for us to change the driver in anyway.

      Acceptance Criteria

      Implementation Requirements

      • functional reqs, potential snafus to avoid, performance targets, etc

      Testing Requirements

      • unit test, spec test sync, etc

      Documentation Requirements

      • DOCSP ticket, API docs, etc

      Follow Up Requirements

      • Change ticket to compass changes needed if we proceed with removing the aws4 requirement for aws auth, so they can remove it from their packaging as well.

            Assignee:
            neal.beeken@mongodb.com Neal Beeken
            Reporter:
            neal.beeken@mongodb.com Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: