improve skip logic

Author: emadum

lib/core/uri_parser.js

@@ -11,6 +11,11 @@ const ReadPreference = require('./topologies/read_preference');
  */
 const HOSTS_RX = /(mongodb(?:\+srv|)):\/\/(?: (?:[^:]*) (?: : ([^@]*) )? @ )?([^/?]*)(?:\/|)(.*)/;
 
+// Options that reference file paths should not be parsed
+const FILE_PATH_OPTIONS = new Set(
+  ['sslCA', 'sslCert', 'sslKey', 'tlsCAFile', 'tlsCertificateKeyFile'].map(key => key.toLowerCase())
+);
+
 /**
  * Determines whether a provided address matches the provided parent domain in order
  * to avoid certain attack vectors.
@@ -424,9 +429,9 @@ function parseQueryString(query, options) {
     }
 
     const normalizedKey = key.toLowerCase();
-    // skip parsing for tls/ssl values to avoid issues with Windows file paths (containing colons)
-    const skipParse = /^(?:ssl|tls)/.test(normalizedKey);
-    const parsedValue = skipParse ? value : parseQueryStringItemValue(normalizedKey, value);
+    const parsedValue = FILE_PATH_OPTIONS.has(normalizedKey)
+      ? value
+      : parseQueryStringItemValue(normalizedKey, value);
     applyConnectionStringOption(result, normalizedKey, parsedValue, options);
   }