Skip to content

Fix Discord.bio and interpals false positives - #3080

Open
Dev9269 wants to merge 1 commit into
sherlock-project:masterfrom
Dev9269:fix/exclusions
Open

Dev9269 wants to merge 1 commit into
sherlock-project:masterfrom
Dev9269:fix/exclusions

Conversation

@Dev9269

@Dev9269 Dev9269 commented Aug 17, 2026

Copy link
Copy Markdown

References: #2547

Changes

Two excluded sites now return clean, detectable responses:

  • Discord.bio — the API returns 404 with a JSON body for missing users (the old Server Error (500) page is gone). Switched to status_code detection with errorCode: 404.
  • interpals — missing users return 404, but the page body is empty so the message-based check no longer matches. Switched to status_code detection, forcing GET since the site loops redirects on HEAD.

Verification

  • Fake usernames: both sites now report not-found (no false positive)
  • Claimed username
    obert/�lue: both still reported as found
  • ests/test_manifest.py passes

- Discord.bio: API now returns 404 + JSON error for missing users
  instead of the old 500 error page; switch to status_code detection.
- interpals: returns 404 for missing users but a message-based check
  no longer matches the empty body; switch to status_code detection
  with GET to avoid an infinite redirect loop on HEAD.
@github-actions

Copy link
Copy Markdown
Contributor

Automatic validation of changes

Target F+ Check F- Check
Discord.bio ✔️   Pass ✔️   Pass
interpals ✔️   Pass ✔️   Pass

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.

1 participant