Skip to content

Return proper JSON 429 error for throttle and local rate limits in service mode#2226

Merged
amangalampalli-ks merged 1 commit into
fix/service-throttle-json-429from
fix/service-throttle-json-429-int
Jul 20, 2026
Merged

Return proper JSON 429 error for throttle and local rate limits in service mode#2226
amangalampalli-ks merged 1 commit into
fix/service-throttle-json-429from
fix/service-throttle-json-429-int

Conversation

@amangalampalli-ks

Copy link
Copy Markdown
Contributor

Summary

Service mode now returns clear JSON HTTP 429 responses for Keeper throttling and the local Flask rate limiter, matching Commander’s retry behavior (fail_on_throttle left off).

Changes

  • Map Keeper throttled / edge 429 and service rate limits to JSON 429 with result_code of throttled, 429, or rate_limited
  • Strip retry-log noise from error messages
  • Add unit tests and README notes for the 429 contract

@amangalampalli-ks amangalampalli-ks self-assigned this Jul 17, 2026
@amangalampalli-ks
amangalampalli-ks marked this pull request as ready for review July 17, 2026 10:27
@amangalampalli-ks
amangalampalli-ks force-pushed the fix/service-throttle-json-429 branch from 0da055d to 040221b Compare July 20, 2026 12:35
@amangalampalli-ks
amangalampalli-ks force-pushed the fix/service-throttle-json-429-int branch from 451bd1b to d7f4c67 Compare July 20, 2026 12:35
@amangalampalli-ks
amangalampalli-ks merged commit 65406cf into fix/service-throttle-json-429 Jul 20, 2026
4 checks passed
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.

3 participants