Draft
Conversation
tricky42
requested changes
Mar 5, 2026
jhickmanit
reviewed
Mar 6, 2026
| ## Rate limits per bucket | ||
|
|
||
| Buckets follow a `{service}-{access}-{cost}` naming pattern. For example: | ||
|
|
Contributor
There was a problem hiding this comment.
I think I will echo what Patrik put into the code PR for this. I am not sure if customers will understand or care about the cost bit. When they simply see something like kratos-public-low they will think it is requests that have a low ratelimit ceiling rather than low cost. I hate to derail this, but we might want to think about how to better label this for the public.
jhickmanit
reviewed
Mar 6, 2026
|
|
||
| - **Burst limit**: Maximum requests per second (rps), allowing for short traffic spikes. | ||
| - **Sustained limit**: Maximum requests per minute (rpm), ensuring consistent performance over time. | ||
|
|
Contributor
There was a problem hiding this comment.
Should we split the technical below into another document? It feels odd to get into the "how" when we are explaining the why above.
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.
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact security@ory.com) from the maintainers to push the changes.
Further comments