fix(data): update InterPals absence string to match current site response#2442
Merged
soxoj merged 1 commit intosoxoj:mainfrom Apr 3, 2026
Merged
Conversation
Owner
|
Thanks @juliosuas! Could you please resolve the data.json conflict? I see that you have added two other sites in this commit. They are in the DB already: https://github.com/soxoj/maigret/blob/main/sites.md |
…onse The previous absence string 'The requested user does not exist or is inactive' no longer matches the live site response. InterPals now returns 'User not found' for non-existent profiles, causing false positives for all username searches. Tested against interpals.net/noneownsthisusername (non-existent) and interpals.net/blue (claimed) to confirm detection accuracy. Closes soxoj#2433
juliosuas
force-pushed
the
fix/false-positive-interpals
branch
from
1fc7a23 to
d48037c
Compare
Contributor
Author
|
Rebased on upstream main and resolved the conflict. Removed the duplicate site entries (infopps.moy.su, iXBT, and the lowercase interpals duplicate) that were already in the database — they were carried over from a previous feature branch merge. The PR now contains only the targeted InterPals absence string fix ( |
Owner
|
Good job 🤝 |
soxoj
pushed a commit
that referenced
this pull request
Apr 7, 2026
…onse (#2442) The previous absence string 'The requested user does not exist or is inactive' no longer matches the live site response. InterPals now returns 'User not found' for non-existent profiles, causing false positives for all username searches. Tested against interpals.net/noneownsthisusername (non-existent) and interpals.net/blue (claimed) to confirm detection accuracy. Closes #2433 Co-authored-by: Julio César Suástegui <juliosuas@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
InterPals was reporting false positives — claiming users exist when they do not (issue #2433).
The site returns HTTP 200 for both existing and non-existing users, so detection relies entirely on page content matching.
Root Cause
The previous absence string
The requested user does not exist or is inactiveno longer matches what interpals.net returns for non-existent profiles.Fix
Updated
absenceStrsto match the current site response:Testing
Manually verified against live site:
interpals.net/noneownsthisusername→ returns "User not found" ✓ (correctly detected as absent)interpals.net/blue→ returns user profile ✓ (correctly detected as claimed)Fixes #2433