Skip to content

Conversation

@amCap1712
Copy link
Member

  • Add optional parameters to ratelimit decorator: per_token_limit, per_ip_limit, window, and scope
  • Add scope parameter to customize rate limits for a bundle of endpoints
  • Implement limit resolution order: decorator params > scope cache > global

@amCap1712 amCap1712 requested a review from mayhem December 10, 2025 19:36
- Add optional parameters to ratelimit decorator: per_token_limit,
  per_ip_limit, window, and scope
- Add scope parameter to customize rate limits for a bundle of endpoints
- Implement limit resolution order: decorator params > scope cache > global
Copy link
Member

@mayhem mayhem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I love the fine grained control, even if I dont understand scopes. :)

@amCap1712 amCap1712 merged commit 331f806 into master Dec 15, 2025
1 check passed
@amCap1712 amCap1712 deleted the ratelimit-2 branch December 15, 2025 12:06
@amCap1712
Copy link
Member Author

Ah, after merging I realised I forgot to push a commit. Will open another PR to fix it.

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