Skip to content

fix: correct false positive for GeeksForGeeks (#2782)#2917

Open
victoireantoine1707-design wants to merge 1 commit into
sherlock-project:masterfrom
victoireantoine1707-design:fix/false-positive-geeksforgeeks
Open

fix: correct false positive for GeeksForGeeks (#2782)#2917
victoireantoine1707-design wants to merge 1 commit into
sherlock-project:masterfrom
victoireantoine1707-design:fix/false-positive-geeksforgeeks

Conversation

@victoireantoine1707-design
Copy link
Copy Markdown

Site redirects to /user/false for non-existent usernames. Changed errorType from status_code to response_url.

Site redirects to /user/false for non-existent usernames.
Changed errorType from status_code to response_url.
@github-actions
Copy link
Copy Markdown
Contributor

Automatic validation of changes

Target F+ Check F- Check
GeeksforGeeks ✔️   Pass ❌   Fail

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

@victoireantoine1707-design
Copy link
Copy Markdown
Author

Manually verified from browser: non-existent usernames redirect to /user/false. Validation failure may be due to regional differences in site behavior.

@ppfeister
Copy link
Copy Markdown
Member

ppfeister commented May 7, 2026

Current behavior on master appears to function as intended (there was a recent merge) --- manual validation of PR from NA/US appears to lead to a F- -- please confirm behavior

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants