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

SDAM should remove servers that have a different setName and set serverDescription fields to null on Unknown servers

    • Type: Icon: Bug Bug
    • Resolution: Fixed
    • Priority: Icon: Minor - P4 Minor - P4
    • 4.9.0
    • Affects Version/s: None
    • Component/s: None
    • 3
    • Not Needed

      What problem are you facing?

      The driver is marking servers Unknown that have a different setName from the one provided in the connection string. According to the specification we should be removing servers altogether.

      Fields in the SDAM unit tests are set to null and the runner attempts to assert that the nullish keys do not exist on the serverDescription. The assertion was written incorrectly so it doesn't prevent these "omittedFields" from existing. We should set these fields to null, and the runner can look for null equality on these fields.

      Steps to reproduce?

      Run this test: test/spec/server-discovery-and-monitoring/rs/new_primary_wrong_set_name.json

      Attempt to assert that the number of servers present in the test file are equal to the number of servers that the Topology has saved. (search code for this ticket number)

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

              Created:
              Updated:
              Resolved: