Configurable DNS domain validation for SRV records: clarifications

ExportXMLWordPrintableJSON

    • Needed - No Spec Changes
    • Hide

      Summary of necessary driver changes

      • srvHostValidator must strictly accept boolean values (and not just truthy or false-y values) 
      • clarify validation of srvAllowedHostsSuffix

      Commits for syncing spec/prose tests
      (and/or refer to an existing language POC if needed)

      Context for other referenced/linked tickets

      Show
      Summary of necessary driver changes srvHostValidator must strictly accept boolean values (and not just truthy or false-y values)  clarify validation of srvAllowedHostsSuffix Commits for syncing spec/prose tests (and/or refer to an existing language POC if needed) two new prose tests: https://github.com/mongodb/specifications/commit/3a0ed901fd95ab2dc17a2652c33ce8779b7fb4e1 Context for other referenced/linked tickets python: https://github.com/mongodb/mongo-python-driver/pull/3084
    • $i18n.getText("admin.common.words.hide")
      Key Status/Resolution FixVersion
      CDRIVER-6489 Backlog
      CXX-3596 Backlog
      CSHARP-6246 Investigating
      GODRIVER-4173 Backlog
      JAVA-6326 Blocked
      NODE-7873 Duplicate
      PYTHON-6134 Fixed 4.19.0
      PHPLIB-1968 Needs Triage
      RUBY-3971 Backlog
      RUST-2516 Fixed 3.10.0
      $i18n.getText("admin.common.words.show")
      #scriptField, #scriptField *{ border: 1px solid black; } #scriptField{ border-collapse: collapse; } #scriptField td { text-align: center; /* Center-align text in table cells */ } #scriptField td.key { text-align: left; /* Left-align text in the Key column */ } #scriptField a { text-decoration: none; /* Remove underlines from links */ border: none; /* Remove border from links */ } /* Add green background color to cells with FixVersion */ #scriptField td.hasFixVersion { background-color: #00FF00; /* Green color code */ } #scriptField td.willNotDo { background-color: #FF0000; /* Red color code */ } /* Center-align the first row headers */ #scriptField th { text-align: center; } Key Status/Resolution FixVersion CDRIVER-6489 Backlog CXX-3596 Backlog CSHARP-6246 Investigating GODRIVER-4173 Backlog JAVA-6326 Blocked NODE-7873 Duplicate PYTHON-6134 Fixed 4.19.0 PHPLIB-1968 Needs Triage RUBY-3971 Backlog RUST-2516 Fixed 3.10.0

      Summary

      The spec for "Configurable DNS domain validation for SRV records" (https://jira.mongodb.org/browse/DRIVERS-3632) does not fully answer the following 2 questions:
      1. How should the Driver respond if the type of use callback is not the expected type? For static languages, this is not possible, but for Node it's entirely possible that the user will pass in a constant expression, or a callback that returns the wrong type. How should this issue be surfaced to the user?
      2. srvAllowedHostsSuffix is described as "a valid DNS hostname suffix", but there is no RFC specified, so different Drivers have implemented this differently. Java implemented RFC 1035 (which does not allow underscores), while Python/Node/C# are going against RFC 2181 (which does allow underscores). This difference in implementations means that Java Driver users may be unable to connect to servers that other Driver are able to connect to.

      Motivation

      Who is the affected end user?

      Drivers maintainers.

      How does this affect the end user?

      Will be confused when some servers are unreachable in broken Drivers.

      How likely is it that this problem or use case will occur?

      Edge case.

      If the problem does occur, what are the consequences and how severe are they?

      Impacted Driver won't be able to connect to some hosts.

      Is this issue urgent?

      No timeline.

      Is this ticket required by a downstream team?

      No.

      Is this ticket only for tests?

      No, spec update as well.

      Acceptance Criteria

      Both issues are addressed in the spec:
      1. Error behavior when the callback is of the wrong type
      2. Specific RFC chosen for DNS host suffix validation

            Assignee:
            Iris Ho
            Reporter:
            Pavel Safronov
            Jib Adegunloye Jib Adegunloye
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: