Skip to content

Commit 2b5cba8

Browse files
committed
JS: Accept some new alerts in HardcodedCredentials
I think these were just missing 'NOT OK' comments
1 parent 2cc194f commit 2b5cba8

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

javascript/ql/test/query-tests/Security/CWE-798/HardcodedCredentials.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -127,12 +127,12 @@
127127
})();
128128

129129
(function(){
130-
require('crypto').createHmac('sha256', 'hgfedcba');
131-
require("crypto-js/aes").encrypt('my message', 'hgfedcba');
130+
require('crypto').createHmac('sha256', 'hgfedcba'); // $ Alert
131+
require("crypto-js/aes").encrypt('my message', 'hgfedcba'); // $ Alert
132132
})()
133133

134134
(function(){
135-
require("cookie-session")({ secret: "hgfedcba" });
135+
require("cookie-session")({ secret: "hgfedcba" }); // $ Alert
136136
})()
137137

138138
(function(){

0 commit comments

Comments
 (0)