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

fix: implemented rate limiting for all requests #585

Merged
merged 2 commits into from
Feb 5, 2025
Merged

Conversation

spacecowboy
Copy link
Owner

@spacecowboy spacecowboy commented Feb 5, 2025

  • Respects 429 responses
  • Limits image loading concurrency to one per host
  • Limits requests to the same host to 3 per second
  • More aggressive caching of images (e.g. always use cached version if available)

fixes #582

Signed-off-by: Jonas Kalderstam <jonas@cowboyprogrammer.org>
Signed-off-by: Jonas Kalderstam <jonas@cowboyprogrammer.org>
@spacecowboy spacecowboy marked this pull request as ready for review February 5, 2025 19:56
@spacecowboy spacecowboy merged commit 4f4107e into master Feb 5, 2025
3 checks passed
@spacecowboy spacecowboy deleted the rate-limit branch February 5, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Excessive requests for the feed icon up to 320 requests in 40 seconds
1 participant