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

remove crypto callbacks from the driver

    • 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 part of https://github.com/mongodb-js/mongodb-client-encryption/pull/25, our FLE bindings now contain C++ crypto callbacks that are automatically loaded into the FLE bindings. These are not available on all supported runtimes and configurations, so the existing crypto callbacks must continue to exist.

      Now that there are C++ crypto callbacks in mongodb-client-encryption, it makes sense to move the JS crypto callbacks there as well so that both implementations are co-located (Anna's PR does this).

      Now that all crypto callbacks are present in mongodb-client-encryption, we can remove the JS crypto callbacks file from the driver.

      User Impact

      • Depends. If we do this in a minor, users wouldn't be able to use `mongodb-client-encryption@6.0.0 with driver@6.9.0+ (or whatever version this ticket is released in). If we wait until the next major, there is none.

      Dependencies

      n/a.

      Unknowns

      • Can we make this change in a minor?

      Acceptance Criteria

      Implementation Requirements

      • remove crypto_callbacks.ts and its associated tests from the driver.

      Testing Requirements

      n/a.

      Documentation Requirements

      n/a.

      Follow Up Requirements

      n/a.

            Assignee:
            Unassigned Unassigned
            Reporter:
            bailey.pearson@mongodb.com Bailey Pearson
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: