Treat 429 and status 6 throttle responses as server errors that respe…#21
Merged
luke-owen-crowdhandler merged 1 commit intoJun 11, 2026
Merged
Conversation
…ct FAIL_TRUST Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the Cloudflare Worker CrowdHandler integration to treat API throttling conditions as server-error equivalents, so the FAIL_TRUST setting is respected during throttling (avoiding unnecessary safety-net redirects when fail-trusting is enabled).
Changes:
- Handle CrowdHandler API “throttle” response body status
6as a server-error case in the request decision flow. - Treat HTTP
429 Too Many Requestsfrom the API as a server-error path (instead of a 4xx client error path) soFAIL_TRUSTbehavior applies. - Refresh committed build artifacts under
dist/to reflect the updated behavior.
Reviewed changes
Copilot reviewed 2 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| index.js | Adds explicit handling for API throttle body status 6 with FAIL_TRUST-aware behavior. |
| helpers/http.js | Classifies HTTP 429 as a server-error-equivalent (fail-trust respected) instead of a client error. |
| dist/index.js | Rebuilt output mirroring the new throttle handling logic. |
| dist/index.js.map | Updated sourcemap for rebuilt output. |
| dist/README.md | Updates build timestamp metadata for the dist/ output. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
luke-owen-crowdhandler
deleted the
maintenance/throttle-special-treatment
branch
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
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.
…ct FAIL_TRUST