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

Request queue management #27

Open
Dentosal opened this issue Dec 8, 2022 · 0 comments
Open

Request queue management #27

Dentosal opened this issue Dec 8, 2022 · 0 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@Dentosal
Copy link
Member

Dentosal commented Dec 8, 2022

Currently we rate limit dispensing to 20 events per minute. If the amount of request exceeds that, the rest of the requests are queued. In the current implementation, the user is not shown any indication of this, and the website simply seems to do nothing. We should at least show the user that they are in queue to receive the tokens. Since we know how long the queue is, that could be displayed as well. However, we likely shouldn't let the queue to grow too large, since the browser will timeout the request if it's waiting too long for a response, typically after 1-3 minutes.

@Dentosal Dentosal added bug Something isn't working enhancement New feature or request labels Dec 8, 2022
@Dentosal Dentosal self-assigned this Dec 8, 2022
@Dentosal Dentosal removed their assignment Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant