DRIVERS-2234 Run a ping after initializing a MongoClient in DNS seedlist tests. (#1173)

matthewdale · kevinAlbs · web-flow · commit b2e35cb9e19d · 2022-08-30T20:43:24.000-07:00
Co-authored-by: Kevin Albertson &lt;kevin.albertson@10gen.com&gt;
diff --git a/source/initial-dns-seedlist-discovery/tests/README.rst b/source/initial-dns-seedlist-discovery/tests/README.rst
@@ -114,31 +114,51 @@ These YAML and JSON files contain the following fields:
 - ``error``: indicates that the parsing of the URI, or the resolving or
   contents of the SRV or TXT records included errors.
 - ``comment``: a comment to indicate why a test would fail.
+- ``ping``: if false, the test runner should not run a "ping" operation.
 
 .. _`Connection String`: ../../connection-string/connection-string-spec.rst
 .. _`URI options`: ../../uri-options/uri-options.rst
 
-For each file, create a MongoClient initialized with the ``mongodb+srv``
-connection string.
-
-If ``seeds`` is specified, drivers SHOULD verify that the set of hosts in the
-client's initial seedlist matches the list in ``seeds``. If ``numSeeds`` is
-specified, drivers SHOULD verify that the size of that set matches ``numSeeds``.
-
-If ``hosts`` is specified, drivers MUST verify that the set of
-ServerDescriptions in the client's TopologyDescription eventually matches the
-list in ``hosts``. If ``numHosts`` is specified, drivers MUST verify that the
-size of that set matches ``numHosts``.
-
-If ``options`` is specified, drivers MUST verify each of the values under
-``options`` match the MongoClient's parsed value for that option. There may be
-other options parsed by the MongoClient as well, which a test does not verify.
-
-If ``parsed_options`` is specified, drivers MUST verify that each of the values
-under ``parsed_options`` match the MongoClient's parsed value for that option.
-Supported values include, but are not limited to, ``user`` and ``password``
-(parsed from ``UserInfo``) and ``auth_database`` (parsed from
-``Auth database``).
-
-If ``error`` is specified and ``true``, drivers MUST verify that an error has
-been thrown.
+For each YAML file:
+
+- Create a MongoClient initialized with the ``mongodb+srv``
+  connection string.
+- Run a "ping" operation unless ``ping`` is false or ``error`` is true.
+
+Assertions:
+
+- If ``seeds`` is specified, drivers SHOULD verify that the set of hosts in the
+  client's initial seedlist matches the list in ``seeds``. If ``numSeeds`` is
+  specified, drivers SHOULD verify that the size of that set matches
+  ``numSeeds``.
+
+- If ``hosts`` is specified, drivers MUST verify that the set of
+  ServerDescriptions in the client's TopologyDescription eventually matches the
+  list in ``hosts``. If ``numHosts`` is specified, drivers MUST verify that the
+  size of that set matches ``numHosts``.
+
+- If ``options`` is specified, drivers MUST verify each of the values under
+  ``options`` match the MongoClient's parsed value for that option. There may be
+  other options parsed by the MongoClient as well, which a test does not verify.
+
+- If ``parsed_options`` is specified, drivers MUST verify that each of the
+  values under ``parsed_options`` match the MongoClient's parsed value for that
+  option. Supported values include, but are not limited to, ``user`` and
+  ``password`` (parsed from ``UserInfo``) and ``auth_database`` (parsed from
+  ``Auth database``).
+
+- If ``error`` is specified and ``true``, drivers MUST verify that initializing
+  the MongoClient throws an error. If ``error`` is not specified or is
+  ``false``, both initializing the MongoClient and running a ping operation must
+  succeed without throwing any errors.
+
+- If ``ping`` is not specified or ``true``, drivers MUST verify that running a
+  "ping" operation using the initialized MongoClient succeeds. If ``ping`` is
+  ``false``, drivers MUST NOT run a "ping" operation.
+
+    **Note:** These tests are expected to be run against MongoDB databases with
+    and without authentication enabled. The "ping" operation does not require
+    authentication so should succeed with URIs that contain no userinfo (i.e.
+    no username and password). Tests with URIs that contain userinfo always set
+    ``ping`` to ``false`` because some drivers will fail handshake on a
+    connection if userinfo is provided but incorrect.
diff --git a/source/initial-dns-seedlist-discovery/tests/load-balanced/loadBalanced-directConnection.json b/source/initial-dns-seedlist-discovery/tests/load-balanced/loadBalanced-directConnection.json
@@ -10,5 +10,6 @@
     "loadBalanced": true,
     "ssl": true,
     "directConnection": false
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/load-balanced/loadBalanced-directConnection.yml b/source/initial-dns-seedlist-discovery/tests/load-balanced/loadBalanced-directConnection.yml
@@ -11,3 +11,4 @@ options:
     loadBalanced: true
     ssl: true
     directConnection: false
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/load-balanced/loadBalanced-true-txt.json b/source/initial-dns-seedlist-discovery/tests/load-balanced/loadBalanced-true-txt.json
@@ -9,5 +9,6 @@
   "options": {
     "loadBalanced": true,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/load-balanced/loadBalanced-true-txt.yml b/source/initial-dns-seedlist-discovery/tests/load-balanced/loadBalanced-true-txt.yml
@@ -8,3 +8,4 @@ hosts:
 options:
     loadBalanced: true
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/load-balanced/srvMaxHosts-zero-txt.json b/source/initial-dns-seedlist-discovery/tests/load-balanced/srvMaxHosts-zero-txt.json
@@ -10,5 +10,6 @@
     "loadBalanced": true,
     "srvMaxHosts": 0,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/load-balanced/srvMaxHosts-zero-txt.yml b/source/initial-dns-seedlist-discovery/tests/load-balanced/srvMaxHosts-zero-txt.yml
@@ -8,3 +8,4 @@ options:
     loadBalanced: true
     srvMaxHosts: 0
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/load-balanced/srvMaxHosts-zero.json b/source/initial-dns-seedlist-discovery/tests/load-balanced/srvMaxHosts-zero.json
@@ -10,5 +10,6 @@
     "loadBalanced": true,
     "srvMaxHosts": 0,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/load-balanced/srvMaxHosts-zero.yml b/source/initial-dns-seedlist-discovery/tests/load-balanced/srvMaxHosts-zero.yml
@@ -8,3 +8,4 @@ options:
     loadBalanced: true
     srvMaxHosts: 0
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/direct-connection-false.json b/source/initial-dns-seedlist-discovery/tests/replica-set/direct-connection-false.json
@@ -11,5 +11,6 @@
   "options": {
     "ssl": true,
     "directConnection": false
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/direct-connection-false.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/direct-connection-false.yml
@@ -8,3 +8,4 @@ hosts:
 options:
     ssl: true
     directConnection: false
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/encoded-userinfo-and-db.json b/source/initial-dns-seedlist-discovery/tests/replica-set/encoded-userinfo-and-db.json
@@ -17,5 +17,6 @@
     "password": "$4to@L8=MC",
     "db": "mydb?"
   },
+  "ping": false,
   "comment": "Encoded user, pass, and DB parse correctly"
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/encoded-userinfo-and-db.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/encoded-userinfo-and-db.yml
@@ -12,4 +12,8 @@ parsed_options:
     user: "b*b@f3tt="
     password: "$4to@L8=MC"
     db: "mydb?"
+# Don't run a ping for URIs that include userinfo. Ping doesn't require authentication, so missing
+# userinfo isn't a problem, but some drivers will fail handshake on a connection if userinfo is
+# provided but incorrect.
+ping: false
 comment: Encoded user, pass, and DB parse correctly
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/loadBalanced-false-txt.json b/source/initial-dns-seedlist-discovery/tests/replica-set/loadBalanced-false-txt.json
@@ -11,5 +11,6 @@
   "options": {
     "loadBalanced": false,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/loadBalanced-false-txt.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/loadBalanced-false-txt.yml
@@ -8,3 +8,4 @@ hosts:
 options:
     loadBalanced: false
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/longer-parent-in-return.json b/source/initial-dns-seedlist-discovery/tests/replica-set/longer-parent-in-return.json
@@ -12,5 +12,6 @@
     "replicaSet": "repl0",
     "ssl": true
   },
+  "ping": true,
   "comment": "Is correct, as returned host name shared the URI root \"test.build.10gen.cc\"."
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/longer-parent-in-return.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/longer-parent-in-return.yml
@@ -8,4 +8,5 @@ hosts:
 options:
     replicaSet: repl0
     ssl: true
+ping: true
 comment: Is correct, as returned host name shared the URI root "test.build.10gen.cc".
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/one-result-default-port.json b/source/initial-dns-seedlist-discovery/tests/replica-set/one-result-default-port.json
@@ -11,5 +11,6 @@
   "options": {
     "replicaSet": "repl0",
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/one-result-default-port.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/one-result-default-port.yml
@@ -8,3 +8,4 @@ hosts:
 options:
     replicaSet: repl0
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/one-txt-record-multiple-strings.json b/source/initial-dns-seedlist-discovery/tests/replica-set/one-txt-record-multiple-strings.json
@@ -11,5 +11,6 @@
   "options": {
     "replicaSet": "repl0",
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/one-txt-record-multiple-strings.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/one-txt-record-multiple-strings.yml
@@ -8,3 +8,4 @@ hosts:
 options:
     replicaSet: repl0
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/one-txt-record.json b/source/initial-dns-seedlist-discovery/tests/replica-set/one-txt-record.json
@@ -12,5 +12,6 @@
     "replicaSet": "repl0",
     "authSource": "thisDB",
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/one-txt-record.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/one-txt-record.yml
@@ -9,3 +9,4 @@ options:
     replicaSet: repl0
     authSource: thisDB
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srv-service-name.json b/source/initial-dns-seedlist-discovery/tests/replica-set/srv-service-name.json
@@ -12,5 +12,6 @@
   "options": {
     "ssl": true,
     "srvServiceName": "customname"
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srv-service-name.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/srv-service-name.yml
@@ -9,3 +9,4 @@ hosts:
 options:
     ssl: true
     srvServiceName: "customname"
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-equal_to_srv_records.json b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-equal_to_srv_records.json
@@ -13,5 +13,6 @@
   "options": {
     "srvMaxHosts": 2,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-equal_to_srv_records.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-equal_to_srv_records.yml
@@ -14,3 +14,4 @@ hosts:
 options:
     srvMaxHosts: 2
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-greater_than_srv_records.json b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-greater_than_srv_records.json
@@ -12,5 +12,6 @@
   "options": {
     "srvMaxHosts": 3,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-greater_than_srv_records.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-greater_than_srv_records.yml
@@ -13,3 +13,4 @@ hosts:
 options:
     srvMaxHosts: 3
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-less_than_srv_records.json b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-less_than_srv_records.json
@@ -9,5 +9,6 @@
   "options": {
     "srvMaxHosts": 1,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-less_than_srv_records.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-less_than_srv_records.yml
@@ -13,3 +13,4 @@ hosts:
 options:
     srvMaxHosts: 1
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-zero-txt.json b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-zero-txt.json
@@ -13,5 +13,6 @@
     "replicaSet": "repl0",
     "srvMaxHosts": 0,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-zero-txt.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-zero-txt.yml
@@ -13,3 +13,4 @@ options:
     replicaSet: repl0
     srvMaxHosts: 0
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-zero.json b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-zero.json
@@ -13,5 +13,6 @@
     "replicaSet": "repl0",
     "srvMaxHosts": 0,
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-zero.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/srvMaxHosts-zero.yml
@@ -13,3 +13,4 @@ options:
     replicaSet: repl0
     srvMaxHosts: 0
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/two-results-default-port.json b/source/initial-dns-seedlist-discovery/tests/replica-set/two-results-default-port.json
@@ -12,5 +12,6 @@
   "options": {
     "replicaSet": "repl0",
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/two-results-default-port.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/two-results-default-port.yml
@@ -9,3 +9,4 @@ hosts:
 options:
     replicaSet: repl0
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/two-results-nonstandard-port.json b/source/initial-dns-seedlist-discovery/tests/replica-set/two-results-nonstandard-port.json
@@ -12,5 +12,6 @@
   "options": {
     "replicaSet": "repl0",
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/two-results-nonstandard-port.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/two-results-nonstandard-port.yml
@@ -9,3 +9,4 @@ hosts:
 options:
     replicaSet: repl0
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/txt-record-with-overridden-ssl-option.json b/source/initial-dns-seedlist-discovery/tests/replica-set/txt-record-with-overridden-ssl-option.json
@@ -12,5 +12,6 @@
     "replicaSet": "repl0",
     "authSource": "thisDB",
     "ssl": false
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/txt-record-with-overridden-ssl-option.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/txt-record-with-overridden-ssl-option.yml
@@ -9,3 +9,4 @@ options:
     replicaSet: repl0
     authSource: thisDB
     ssl: false
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/txt-record-with-overridden-uri-option.json b/source/initial-dns-seedlist-discovery/tests/replica-set/txt-record-with-overridden-uri-option.json
@@ -12,5 +12,6 @@
     "replicaSet": "repl0",
     "authSource": "otherDB",
     "ssl": true
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/txt-record-with-overridden-uri-option.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/txt-record-with-overridden-uri-option.yml
@@ -9,3 +9,4 @@ options:
     replicaSet: repl0
     authSource: otherDB
     ssl: true
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/uri-with-admin-database.json b/source/initial-dns-seedlist-discovery/tests/replica-set/uri-with-admin-database.json
@@ -15,5 +15,6 @@
   },
   "parsed_options": {
     "auth_database": "adminDB"
-  }
+  },
+  "ping": true
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/uri-with-admin-database.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/uri-with-admin-database.yml
@@ -11,3 +11,4 @@ options:
     ssl: true
 parsed_options:
     auth_database: adminDB
+ping: true
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/uri-with-auth.json b/source/initial-dns-seedlist-discovery/tests/replica-set/uri-with-auth.json
@@ -9,9 +9,14 @@
     "localhost:27018",
     "localhost:27019"
   ],
+  "options": {
+    "replicaSet": "repl0",
+    "ssl": true
+  },
   "parsed_options": {
     "user": "auser",
     "password": "apass"
   },
+  "ping": false,
   "comment": "Should preserve auth credentials"
 }
diff --git a/source/initial-dns-seedlist-discovery/tests/replica-set/uri-with-auth.yml b/source/initial-dns-seedlist-discovery/tests/replica-set/uri-with-auth.yml
@@ -6,7 +6,14 @@ hosts:
     - localhost:27017
     - localhost:27018
     - localhost:27019
+options:
+    replicaSet: repl0
+    ssl: true
 parsed_options:
     user: auser
     password: apass
+# Don't run a ping for URIs that include userinfo. Ping doesn't require authentication, so missing
+# userinfo isn't a problem, but some drivers will fail handshake on a connection if userinfo is
+# provided but incorrect.
+ping: false
 comment: Should preserve auth credentials
diff --git a/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-equal_to_srv_records.json b/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-equal_to_srv_records.json
diff --git a/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-equal_to_srv_records.yml b/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-equal_to_srv_records.yml
diff --git a/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-greater_than_srv_records.json b/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-greater_than_srv_records.json
diff --git a/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-greater_than_srv_records.yml b/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-greater_than_srv_records.yml
diff --git a/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-less_than_srv_records.json b/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-less_than_srv_records.json
diff --git a/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-less_than_srv_records.yml b/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-less_than_srv_records.yml
diff --git a/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-zero.json b/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-zero.json
diff --git a/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-zero.yml b/source/initial-dns-seedlist-discovery/tests/sharded/srvMaxHosts-zero.yml

Original file line number	Diff line number	Diff line change
`@@ -10,5 +10,6 @@`
`10`	`10`	`"loadBalanced": true,`
`11`	`11`	`"ssl": true,`
`12`	`12`	`"directConnection": false`
`13`		`- }`
	`13`	`+ },`
	`14`	`+ "ping": true`
`14`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,5 +9,6 @@`
`9`	`9`	`"options": {`
`10`	`10`	`"loadBalanced": true,`
`11`	`11`	`"ssl": true`
`12`		`- }`
	`12`	`+ },`
	`13`	`+ "ping": true`
`13`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,5 +11,6 @@`
`11`	`11`	`"options": {`
`12`	`12`	`"ssl": true,`
`13`	`13`	`"directConnection": false`
`14`		`- }`
	`14`	`+ },`
	`15`	`+ "ping": true`
`15`	`16`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,5 +17,6 @@`
`17`	`17`	`"password": "$4to@L8=MC",`
`18`	`18`	`"db": "mydb?"`
`19`	`19`	`},`
	`20`	`+ "ping": false,`
`20`	`21`	`"comment": "Encoded user, pass, and DB parse correctly"`
`21`	`22`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,5 +12,6 @@`
`12`	`12`	`"replicaSet": "repl0",`
`13`	`13`	`"ssl": true`
`14`	`14`	`},`
	`15`	`+ "ping": true,`
`15`	`16`	`"comment": "Is correct, as returned host name shared the URI root \"test.build.10gen.cc\"."`
`16`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,5 +13,6 @@`
`13`	`13`	`"options": {`
`14`	`14`	`"srvMaxHosts": 2,`
`15`	`15`	`"ssl": true`
`16`		`- }`
	`16`	`+ },`
	`17`	`+ "ping": true`
`17`	`18`	`}`
Original file line number	Diff line number	Diff line change
`@@ -15,5 +15,6 @@`
`15`	`15`	`},`
`16`	`16`	`"parsed_options": {`
`17`	`17`	`"auth_database": "adminDB"`
`18`		`- }`
	`18`	`+ },`
	`19`	`+ "ping": true`
`19`	`20`	`}`