Skip to content

Commit 9ad8a85

Browse files
committed
Delete redundant checks in verifiesSignature()
1 parent 3fe2a08 commit 9ad8a85

File tree

2 files changed

+0
-5
lines changed

2 files changed

+0
-5
lines changed

python/ql/src/experimental/semmle/python/libraries/PyJWT.qll

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,6 @@ private module PyJWT {
8686
}
8787

8888
override predicate verifiesSignature() {
89-
this.hasNoVerifyArgumentOrOptions()
90-
or
9189
not this.hasVerifySetToFalse() and
9290
not this.hasVerifySignatureSetToFalse()
9391
}

python/ql/src/experimental/semmle/python/libraries/PythonJose.qll

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,9 +87,6 @@ private module PythonJose {
8787
}
8888

8989
override predicate verifiesSignature() {
90-
// jwt.decode(token, "key", "HS256")
91-
this.hasNoOptions()
92-
or
9390
// jwt.decode(token, key, options={"verify_signature": False})
9491
not this.hasVerifySignatureSetToFalse()
9592
}

0 commit comments

Comments
 (0)