intrigus-lgtm · GitHub

@intrigus-lgtm

Closed

1 task

Closed

1 task

Closed

1 task

Merged

1 task

@intrigus-lgtm

@intrigus-lgtm

smowton

intrigus-lgtm

intrigus-lgtm

smowton

@felicitymay

This removes the sanitizer part that classified some results as FP
if the results were in methods with certain names, like
`disableVerification()`. I now think that it's a bad idea to filter
based on the method name.
The custom flow steps in `flagFlowStep` are now listed explicitly.
Simplified check whether a method throws an exception.
Previously intentionally unsafe methods such as `disableCertificate`
would be ignored by this query. But now they will also be flagged
as it is hard to guess intentions...
Adjust the tests to account for this change.
Co-authored-by: Anders Schack-Mulligen <aschackmull@users.noreply.github.com>
Co-authored-by: Felicity Chapman <felicitymay@github.com>
Move comments to separate lines to improve
the rendering in the finished query help.

@intrigus-lgtm

@intrigus-lgtm

@smowton

Merged

Merged

Read the original on github.com ↗