fix(MongoClient): only check own properties for valid options

PassTheMayo · daprahamian · commit c9dc7174d6f0 · 2019-08-13T10:00:31.000-04:00
diff --git a/lib/operations/mongo_client_ops.js b/lib/operations/mongo_client_ops.js
@@ -596,7 +596,7 @@ function validOptions(options) {
   const _validOptions = validOptionNames.concat(legacyOptionNames);
 
   for (const name in options) {
-    if (ignoreOptionNames.indexOf(name) !== -1) {
+    if (ignoreOptionNames.indexOf(name) !== -1 || !options.hasOwnProperty(name)) {
       continue;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -596,7 +596,7 @@ function validOptions(options) {`
`596`	`596`	`const _validOptions = validOptionNames.concat(legacyOptionNames);`
`597`	`597`
`598`	`598`	`for (const name in options) {`
`599`		`- if (ignoreOptionNames.indexOf(name) !== -1) {`
	`599`	`+ if (ignoreOptionNames.indexOf(name) !== -1 \|\| !options.hasOwnProperty(name)) {`
`600`	`600`	`continue;`
`601`	`601`	`}`
`602`	`602`