Skip to content
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.

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

fix: ProductHunt false positive #2300

Closed
wants to merge 2 commits into from

Conversation

ReganBell
Copy link
Contributor

Not 100% sure why the errorMessage based detection was failing but just using status_code instead fixes.

Tested on known positives and negatives.

@ppfeister
Copy link
Member

Closing PR due to an annoying github issues relating to conflict resolution, but in a few minutes a commit will be pushed with you as one of the authors. Appreciate the fix @ReganBell !

@ppfeister ppfeister closed this Feb 3, 2025
@ppfeister
Copy link
Member

Ref: af2bb98

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.

2 participants