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

Alter ClientMetadata and mark as internal + remove extendedClientMetadata option from codebase

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Unknown Unknown
    • None
    • Affects Version/s: None
    • Component/s: None
    • 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?

      User Impact

      • Client metadata will now be a promise that is awaited at connect() time 
        • This is because capturing docker metadata is an async process

      Acceptance Criteria

      Implementation Requirements

      • Changes to ClientMetadata
        • Mark ClientMetadata interface as internal
        • Make env?.name property optional
        • Add in  env?.container?.orchestrator, and env?.container?.runtime properties to interface
      • Refactor addContainerMetadata to be inside of makeClientMetadata 
      • Delete extendedClientMetadata property from codebase
        • This is because clientMetadata will now hold container data 
        • await clientMetadata instead of extendedClientMetadata in prepareHandshakeDocument

      Testing Requirements

      • Make sure existing tests pass

      Documentation Requirements

      • Update API for client metadata

            Assignee:
            Unassigned Unassigned
            Reporter:
            aditi.khare@mongodb.com Aditi Khare
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: