-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
startTime property in RateLimiter #752
Labels
Comments
Hi, yes, good idea. |
FYI @storozhukBM |
Hi! I created PR #2100 to fix this issue. PTAL when you have some times~ thanks! 😃 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I propose to add a startTime property to RateLimiterConfig so that we can set from when does a RateLimiting period start. This is necessary if someone wants to user RateLimiter both on his server and client. If you have your clocks in synce then with this property one could make limit period overlap on the server and client.
The text was updated successfully, but these errors were encountered: