Skip to content

feat: improve rate limiter - #2234

Open
Litarnus wants to merge 1 commit into
masterfrom
rate-limiter-fixes
Open

Litarnus wants to merge 1 commit into
masterfrom
rate-limiter-fixes

Conversation

@Litarnus

@Litarnus Litarnus commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Improves how we handle rate limits to align it with the specs and fix subtle bugs:

  1. A shorter duration from a second request could overwrite the longer duration obtained previously. This is a violation of the specification. Always keep the maximum rate limit if multiple rate limits reference the same category. If a new rate limit is shorter than an already stored rate limit, then keep the longer one.
  2. 429 without headers ignored rate limits, now it applies the default.
  3. Improved handling for decimal durations

Reference: https://develop.sentry.dev/sdk/foundations/transport/rate-limiting/

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