JS: remove TODO

esbena · esbena · commit 47d3542cfd7a · 2020-06-18T12:38:18.000+02:00
diff --git a/javascript/ql/src/Security/CWE-730/UnclosedStream.ql b/javascript/ql/src/Security/CWE-730/UnclosedStream.ql
@@ -66,7 +66,7 @@ class Stream extends DataFlow::SourceNode {
     exists(DataFlow::CallNode onData |
       this instanceof EventEmitter and
       onData = this.getAMethodCall(EventEmitter::on()) and
-      onData.getArgument(0).mayHaveStringValue("data") and // TODO generalize beoyond data?
+      onData.getArgument(0).mayHaveStringValue("data") and
       processor = onData.getCallback(1)
     )
   }

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ class Stream extends DataFlow::SourceNode {`
`66`	`66`	`exists(DataFlow::CallNode onData \|`
`67`	`67`	`this instanceof EventEmitter and`
`68`	`68`	`onData = this.getAMethodCall(EventEmitter::on()) and`
`69`		`- onData.getArgument(0).mayHaveStringValue("data") and // TODO generalize beoyond data?`
	`69`	`+ onData.getArgument(0).mayHaveStringValue("data") and`
`70`	`70`	`processor = onData.getCallback(1)`
`71`	`71`	`)`
`72`	`72`	`}`