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

Change host address temporary url string to use http protocol

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Minor - P4 Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: Connection String
    • 1
    • 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... cloud engineer
      I want... driver compatibility with browser URL constructor
      So that... host address returns host string

      When using the URL constructor in browser, the ILoveJS protocol makes the hostname an empty string and port number undefined, resulting in a default assignment of 27017.

      See:
      https://github.com/mongodb-js/mongodb-connection-string-url/pull/1
      https://github.com/mongodb-js/mongodb-connection-string-url/pull/6

      User Impact

      none, changing this does not affect driver users

      Dependencies

      • none

      Unknowns

      • none

      Acceptance Criteria

      Implementation Requirements

      • Use the whatwg-url and not the url polyfill in the driver

      Testing Requirements

      • consider unit test for fake url constructor that replicates bug in browser

      Documentation Requirements

      • none

      Follow Up Requirements

      • none

            Assignee:
            Unassigned Unassigned
            Reporter:
            laurel.xiang@mongodb.com Laurel Xiang
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated: