Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: solver rate limiter exemption list #507

Merged
merged 4 commits into from
Feb 7, 2025
Merged

Conversation

kelindi
Copy link
Contributor

@kelindi kelindi commented Jan 31, 2025

This pull request makes the following changes:

  • Adds a new env variable to add exemptions for the ratelimitter

@kelindi kelindi requested a review from a team as a code owner January 31, 2025 23:43
@cla-bot cla-bot bot added the cla-signed label Jan 31, 2025
@kelindi kelindi changed the title solver rate limitter exemptions solver rate limitter allowlist Jan 31, 2025
@kelindi kelindi changed the title solver rate limitter allowlist solver rate limitter exemption list Jan 31, 2025
@kelindi kelindi changed the title solver rate limitter exemption list feat:solver rate limiter exemption list Feb 3, 2025
@kelindi kelindi changed the title feat:solver rate limiter exemption list feat: solver rate limiter exemption list Feb 3, 2025
kelindi and others added 2 commits February 4, 2025 16:28
* fix: Use httprate.WithErrorHandler to exempt IPs

* added some tests for exempt ips

* test: Update rate limit test
Copy link
Contributor

@bgins bgins left a comment

Choose a reason for hiding this comment

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

Looks good! 🎉

@kelindi kelindi merged commit 66d7ee3 into api-net Feb 7, 2025
3 checks passed
@kelindi kelindi deleted the rate-limitter-exemptions branch February 7, 2025 17:21
kelindi added a commit that referenced this pull request Feb 10, 2025
Adds a way to exempt ips from the solver rate limitter
kelindi added a commit that referenced this pull request Feb 11, 2025
Adds a way to exempt ips from the solver rate limitter
kelindi added a commit that referenced this pull request Feb 11, 2025
Adds a way to exempt ips from the solver rate limitter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants