Skip to content

feat: add rate limiter utility for API endpoints#288

Open
shivv23 wants to merge 2 commits intoc2siorg:mainfrom
shivv23:pr/rate-limiter-utility
Open

feat: add rate limiter utility for API endpoints#288
shivv23 wants to merge 2 commits intoc2siorg:mainfrom
shivv23:pr/rate-limiter-utility

Conversation

@shivv23
Copy link
Copy Markdown
Contributor

@shivv23 shivv23 commented Apr 6, 2026

Add rate limiter utility for API endpoints.

Changes:

  • app/rate_limiter.py: New file with RateLimiter class and middleware

Provides token bucket rate limiting with configurable limits.

@shivv23 shivv23 force-pushed the pr/rate-limiter-utility branch 3 times, most recently from 9e0758b to e48b75f Compare April 11, 2026 18:52
@ivantha
Copy link
Copy Markdown
Member

ivantha commented Apr 13, 2026

This PR's head branch is behind main. Please rebase onto the latest main (e.g. git fetch origin && git rebase origin/main) and force-push to make it mergeable.

Note: the RateLimiter/RateLimitMiddleware are not yet wired into main.py — consider integrating them or tracking that as a follow-up.

@shivv23 shivv23 force-pushed the pr/rate-limiter-utility branch from e48b75f to 62d4cc1 Compare April 14, 2026 05:29
@shivv23
Copy link
Copy Markdown
Contributor Author

shivv23 commented Apr 14, 2026

Thanks @ivantha! The branch is now rebased onto main.Regarding the RateLimiter not being wired into main.py - you're right. This PR adds the rate limiter utility/decorator. Wiring it into the API endpoints will be done in a follow-up PR to keep changes focused.

@github-actions
Copy link
Copy Markdown

PR Review

Rebase

Merge commits detected — please use rebase instead of merge:

bd2aa71 Merge branch 'main' into pr/rate-limiter-utility

Squash

Your PR has 2 commits. Please squash into a single commit.

How to fix

git fetch origin
git rebase -i origin/main   # mark all but first commit as "squash"
git push --force-with-lease

This comment updates automatically on each push.

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