Uploaded image for project: 'Go Driver'
  1. Go Driver
  2. GODRIVER-2056

SRV polling is not spawned because options.MergeClientOptions doesn't handle uri properly

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical - P2
    • Resolution: Fixed
    • Affects Version/s: 1.5.3
    • Fix Version/s: 1.5.4
    • Component/s: Options & Configuration
    • Labels:
      None

      Description

      It seems that SRV polling is not working when using `opts.ApplyURI()` without deprecated deployment options. This made us to restart the application when updating Route53 DNS SRV.

      I've found that `pollSRVRecords` is not running because URI is not passed to topology configuration.

      So `options.MergeClientOptions` must copy URI like errors to correctly determine the polling requirement.

      if t.cfg.uri != "" {
      		 
         t.pollingRequired = strings.HasPrefix(t.cfg.uri, "mongodb+srv://")
      		 
      }

        Attachments

          Issue Links

          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. GODRIVER-2057 SRV polling is not spawned because options.MergeClientOptions doesn't handle uri properly

          • Unknown - Priority not determined yet.
          • Closed
          related to

          Bug - A problem which impairs or prevents the functions of the product. GODRIVER-908 options.MergeClientOptions doesn't handle errors properly

          • Major - P3 - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. GODRIVER-1507 SRV polling not performed if the URI is not passed to the topology

          • Major - P3 - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. GODRIVER-2069 Add srv polling integration test

          • Minor - P4 - Minor loss of function, or other problem where easy workaround is present.
          • Closed

            Activity

              People

              Assignee:
              isabella.siu Isabella Siu (Inactive)
              Reporter:
              harebox@gmail.com Hoonmin Kim
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: