Skip to content

Conversation

@MaxwellOldshein
Copy link
Contributor

@MaxwellOldshein MaxwellOldshein commented Jan 6, 2026

This PR corrects the false positive issue for the GeeksForGeeks site as stated in: #2782. This was due to the fact that there is no longer a reliable way to check valid vs. invalid usernames for the GeeksForGeeks site, which used to leveraged the status_code methodology when it was working.

Known Valid Username Behavior --> adam:

curl -Ls -o /dev/null -w '%{http_code} %{url_effective}\n' \ 'https://auth.geeksforgeeks.org/user/adam' --> 200 https://www.geeksforgeeks.org/profile/adam

Known Invalid Username Behavior --> thisisatotallyfakeusername123456:

curl -Ls -o /dev/null -w '%{http_code} %{url_effective}\n' \ 'https://auth.geeksforgeeks.org/user/thisisatotallyfakeusername123456' --> 200 https://www.geeksforgeeks.org/profile/thisisatotallyfakeusername123456

Closes #2782.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

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).

@MaxwellOldshein MaxwellOldshein marked this pull request as draft January 6, 2026 06:08
@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

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).

@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2026

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).

@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2026

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).

@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2026

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).

@greyhatindustries

This comment was marked as spam.

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.

False positive for: geeksforgeeks

2 participants