Skip to content

Conversation

JLLeitschuh
Copy link
Contributor

@JLLeitschuh JLLeitschuh requested a review from a team as a code owner February 4, 2020 20:41
aschackmull
aschackmull previously approved these changes Feb 5, 2020
Copy link
Contributor

@aschackmull aschackmull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@aschackmull
Copy link
Contributor

There's a compilation error:

[ERROR] Characteristic predicate named InsecureDefaultFullHttpResponseClassInstantiation expected (/home/jenkins/workspace/Language-Tests/Java/job/ql/java/ql/src/Security/CWE/CWE-113/NettyResponseSplitting.ql:33,3-48)

@JLLeitschuh
Copy link
Contributor Author

Fixed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants