Skip to content

Commit b6417ca

Browse files
committed
Java: Fix alert message
The signing key that is being set, is _not_ what is being parsed. A _JWT_ is being parsed, that will then be verified using the set key. (Or in our case not, because we're looking for security problems :P)
1 parent b291ee3 commit b6417ca

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

java/ql/src/Security/CWE/CWE-347/MissingJWTSignatureCheck.ql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,5 @@ import MissingJwtSignatureCheckFlow::PathGraph
1616

1717
from MissingJwtSignatureCheckFlow::PathNode source, MissingJwtSignatureCheckFlow::PathNode sink
1818
where MissingJwtSignatureCheckFlow::flowPath(source, sink)
19-
select sink.getNode(), source, sink, "This parses a $@, but the signature is not verified.",
19+
select sink.getNode(), source, sink, "This sets a $@, but the signature is not verified.",
2020
source.getNode(), "JWT signing key"

0 commit comments

Comments
 (0)