-
Type: Bug
-
Resolution: Fixed
-
Priority: Minor - P4
-
Affects Version/s: None
-
Component/s: None
-
None
In SRV polling, if srvMaxHosts is declared, we currently shuffle the list of parsed hosts returned from t.dnsResolver.ParseHosts and then call processSRVResults on those parsed hosts. Depending on the outcome of the shuffle, that logic can result in accidentally losing an existing SRV record. We should move the shuffling logic for SRV polling into processSRVResults to account for this.
Also, now that we've dropped support for Golang 1.9, we can just use rand.Shuffle here and here as stated in the TODO (I missed this in GODRIVER-1876).
- is related to
-
DRIVERS-1981 Reconsider interaction between srvMaxHosts and SRV polling
- Backlog