Skip to content

Flow 181/rate limit exceeded details#60

Merged
rmustafin merged 3 commits intomasterfrom
FLOW-181/rate_limit_exceeded_details
Jun 25, 2021
Merged

Flow 181/rate limit exceeded details#60
rmustafin merged 3 commits intomasterfrom
FLOW-181/rate_limit_exceeded_details

Conversation

@rmustafin
Copy link
Contributor

Intent (What)

Add some details to RateLimitExceeded, it can help us to be smarter about rescheduling jobs.

"retry-after" - tells us when we can retry request (in seconds)
"x-rate-limit-problem" - it can be minute/day/app (??? I'm not sure about app)
"x-minlimit-remaining" - remaining req/minute limit 
"x-daylimit-remaining" - remaining req/day limit 
"x-appminlimit-remaining" - remaining req/app limit

Motivation (Why)

Implementation (How)

Consequence

@rmustafin rmustafin requested review from a team, chrisrbnelson and rudylee June 23, 2021 13:29
@rmustafin rmustafin marked this pull request as ready for review June 23, 2021 13:29
@rmustafin rmustafin merged commit 43d5243 into master Jun 25, 2021
@rmustafin rmustafin deleted the FLOW-181/rate_limit_exceeded_details branch June 25, 2021 18:30
lankz pushed a commit that referenced this pull request Jun 10, 2025
…details

Flow 181/rate limit exceeded details
lankz pushed a commit that referenced this pull request Jun 10, 2025
…details

Flow 181/rate limit exceeded details
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