Merge pull request #1473 from Annoraaq/patch-1

christkv · web-flow · commit e4f59997a32d · 2017-02-05T21:58:17.000+01:00
Fix sensitive command check
diff --git a/lib/apm.js b/lib/apm.js
@@ -181,7 +181,7 @@ var Instrumentation = function(core, options, callback) {
       };
 
       // Filter out any sensitive commands
-      if(senstiveCommands.indexOf(commandName.toLowerCase())) {
+      if(senstiveCommands.indexOf(commandName.toLowerCase()) != -1) {
         command.commandObj = {};
         command.commandObj[commandName] = true;
       }
@@ -208,7 +208,7 @@ var Instrumentation = function(core, options, callback) {
           command.failure = err || r.result.writeErrors || r.result;
 
           // Filter out any sensitive commands
-          if(senstiveCommands.indexOf(commandName.toLowerCase())) {
+          if(senstiveCommands.indexOf(commandName.toLowerCase()) != -1) {
             command.failure = {};
           }
 

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,7 @@ var Instrumentation = function(core, options, callback) {`
`181`	`181`	`};`
`182`	`182`
`183`	`183`	`// Filter out any sensitive commands`
`184`		`- if(senstiveCommands.indexOf(commandName.toLowerCase())) {`
	`184`	`+ if(senstiveCommands.indexOf(commandName.toLowerCase()) != -1) {`
`185`	`185`	`command.commandObj = {};`
`186`	`186`	`command.commandObj[commandName] = true;`
`187`	`187`	`}`
`@@ -208,7 +208,7 @@ var Instrumentation = function(core, options, callback) {`
`208`	`208`	`command.failure = err \|\| r.result.writeErrors \|\| r.result;`
`209`	`209`
`210`	`210`	`// Filter out any sensitive commands`
`211`		`- if(senstiveCommands.indexOf(commandName.toLowerCase())) {`
	`211`	`+ if(senstiveCommands.indexOf(commandName.toLowerCase()) != -1) {`
`212`	`212`	`command.failure = {};`
`213`	`213`	`}`
`214`	`214`