-
-
Notifications
You must be signed in to change notification settings - Fork 801
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
Add FAQ entry for deleting IPs from rate limiter #2932
base: master
Are you sure you want to change the base?
Conversation
Thanks for submitting this pull request. bors try Note: if this build fails, read this. |
tryBuild failed: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I think we should document that tokens are exempt from rate limits... and remind people of |
good point, I will update this PR soon. |
AUTH_RATELIMIT_EXEMPTION
Pull request has been modified.
bors try |
tryBuild succeeded: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It may be worth mentioning on AUTH_RATELIMIT_USER
that tokens are exempted
use @nextgens suggestion Co-authored-by: Florent Daigniere <[email protected]>
Pull request has been modified.
Thanks for the suggestions, I added them. |
Thanks for submitting this pull request. bors try Note: if this build fails, read this. |
tryBuild succeeded: |
What type of PR?
enhancement of documentation
What does this PR do?
adds an FAQ-entry how to manually delete an IP from rate limiter
Related issue(s)
Prerequisites
Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.