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.

Sign up for GitHub

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

Jump to bottom

Fix false positives #1843

Closed

n-athan wants to merge 3 commits into sherlock-project:master from n-athan:FixFalsePositives

n-athan commented Jul 8, 2023 •

edited

Loading

Several sites returned false positives. After looking into these sites I made some fixes.

Fixed sites:

Quizlet ( closes Quizlet false positive #1837 )
ebio.gg

Removed sites:

AllMyLinks. This also had to be removed out the unit tests.
GunsAndAmmo.com
Enjin.com ( closes Enjin false positive #1777 )
IRL ( closes IRL.com has shut down #1838 )
Authorstream ( closes authorstream false positive #1835 )
jeuxvideo.com (closes Jeuxvideo.com #904 )

= added 3 commits

July 8, 2023 12:46


          fixed false positives

f6aa2a8


          removed site replaces in unit tests

d509665


          cleaned up module I used in testing

0bff1a8

ppfeister mentioned this pull request

Reduce false positives (those not caused by WAFs or bot detection) #2068

Merged

sdushantha closed this in #2068

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet