Uploaded image for project: 'Compass '
  1. Compass
  2. COMPASS-8613

Investigate changes in NODE-6610: Change client feature flags from Symbols to internal strings

    • Type: Icon: Investigation Investigation
    • Resolution: Gone away
    • Priority: Icon: Major - P3 Major - P3
    • No version
    • Affects Version/s: None
    • Component/s: None
    • Not Needed
    • Hide

      Since the @@mdb.decorateDecryptionResult and @@mdb.decryptedKeys symbols were striked from the original ticket, we don't expect the change to affect the Compass codebase.

      Show
      Since the @@mdb.decorateDecryptionResult and @@mdb.decryptedKeys symbols were striked from the original ticket, we don't expect the change to affect the Compass codebase.

      Use Case

      As a... Driver engineer
      I want... to change all internal feature flags from symbols to double-underscored strings
      So that... we can configure the driver for testing with regular json input

      User Experience

      • No change for users; internal change

      Dependencies

      • Changes to the following will affect Compass
        • Symbol.for('@@mdb.decorateDecryptionResult')
        • Symbol.for('@@mdb.decryptedKeys')

      Risks/Unknowns

      • None

      Acceptance Criteria

      Implementation Requirements

      • Replace the following Symbol feature  flags with internal double-underscored properties
        • Symbol.for('@@mdb.skipPinOnConnect')
        • Symbol.for('@@mdb.enableMongoLonger')
        • Symbol.for('@@mdb.internalLoggerConfig')
        • Symbol.for('@@mdb.decorateDecryptionResult')
        • Symbol.for('@@mdb.decryptedKeys')

      Testing Requirements

      • Change instances of the symbol flags to the double-underscored string properties

      Documentation Requirements

      • None

      Follow Up Requirements

      • None

            Assignee:
            kraen.hansen@mongodb.com Kræn Hansen
            Reporter:
            dbeng-pm-bot PM Bot
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: