SERVER-18567 Replace usages of the legacy ConnectionString::parse

This is all the non-sharding code, which is using it.

2 files changed, 12 insertions, 8 deletions

diff --git a/src/mongo/scripting/v8-3.25_db.cpp b/src/mongo/scripting/v8-3.25_db.cpp
index 0c9054314bd..69066dfb2f3 100644
--- a/src/mongo/scripting/v8-3.25_db.cpp
+++ b/src/mongo/scripting/v8-3.25_db.cpp
@@ -139,12 +139,14 @@ namespace mongo {
                 args.IsConstructCall());
         verify(scope->MongoFT()->HasInstance(args.This()));
 
-        string errmsg;
-        ConnectionString cs = ConnectionString::parse(host, errmsg);
-        if (!cs.isValid()) {
-            return v8AssertionException(errmsg);
+        auto statusWithHost = ConnectionString::parse(host);
+        if (!status.isOK()) {
+            return v8AssertionException(statusWithHost.getStatus().reason());
         }
 
+        const ConnectionString cs(statusWithHost.getValue());
+
+        string errmsg;
         DBClientBase* conn;
         conn = cs.connect(errmsg);
         if (!conn) {
diff --git a/src/mongo/scripting/v8_db.cpp b/src/mongo/scripting/v8_db.cpp
index c4b45d29ad9..2f9ae8c604e 100644
--- a/src/mongo/scripting/v8_db.cpp
+++ b/src/mongo/scripting/v8_db.cpp
@@ -145,12 +145,14 @@ namespace mongo {
                 args.IsConstructCall());
         verify(scope->MongoFT()->HasInstance(args.This()));
 
-        string errmsg;
-        ConnectionString cs = ConnectionString::parse(host, errmsg);
-        if (!cs.isValid()) {
-            return v8AssertionException(errmsg);
+        auto statusWithHost = ConnectionString::parse(host);
+        if (!statusWithHost.isOK()) {
+            return v8AssertionException(statusWithHost.getStatus().reason());
         }
 
+        const ConnectionString cs(statusWithHost.getValue());
+
+        string errmsg;
         DBClientBase* conn;
         conn = cs.connect(errmsg);
         if (!conn) {