-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Rate limiter #894
Rate limiter #894
Conversation
c916be5
to
231a466
Compare
96d8b6f
to
e566ea9
Compare
Any progress on accepting & merging this PR? |
I'm planning on reviving the other original PR imminently, but with some additional changes on top based on some minor tweaks we made in a private fork we've been running in production for the last 18 months. |
Thank you for trying to resurrect #666 via this PR @YarekTyshchenko. As this has been in draft for a while now I'm going to close this. I've created a new PR with the aim to get the rate limiting policy merged here - #903. Please address any comments on the functionality there. |
The issue or feature being addressed
This PR is a resurrection of #666 only because the author seems to be unreachable. Most of the work has already been done by @reisenberger
Solves Issue #260
Details on the issue fix or feature implementation
Confirm the following