Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Java: JWT decoding without verification [smowton fork] #17020

Closed
wants to merge 7 commits into from

Conversation

smowton
Copy link
Contributor

@smowton smowton commented Jul 19, 2024

This is a polishing/stabilisation fork of #14089 -- discussion should happen there

Copy link
Contributor

QHelp previews:

java/ql/src/experimental/Security/CWE/CWE-347/Auth0NoVerifier.qhelp

errors/warnings:

/home/runner/work/codeql/codeql/java/ql/src/experimental/Security/CWE/CWE-347/Auth0NoVerifier.qhelp: Invalid link target: CVE-2021-37580
A fatal error occurred: 1 qhelp files could not be processed.

PayloadType() { this.hasQualifiedName("com.auth0.jwt.interfaces", "Payload") }
}

class JWTType extends RefType {

Check warning

Code scanning / CodeQL

Acronyms should be PascalCase/camelCase. Warning

Acronyms in JWTType should be PascalCase/camelCase.
JWTType() { this.hasQualifiedName("com.auth0.jwt", "JWT") }
}

class JWTVerifierType extends RefType {

Check warning

Code scanning / CodeQL

Acronyms should be PascalCase/camelCase. Warning

Acronyms in JWTVerifierType should be PascalCase/camelCase.
*/

import java
import semmle.code.java.dataflow.DataFlow

Check warning

Code scanning / CodeQL

Redundant import Warning

Redundant import, the module is already imported inside
semmle.code.java.dataflow.FlowSources
.
@smowton
Copy link
Contributor Author

smowton commented Aug 21, 2024

Original PR now merged.

@smowton smowton closed this Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants