Skip to content

feat(app): handle rate limits#6790

Merged
Prospector merged 5 commits into
mainfrom
fetch/app-rate-limit-handling
Jul 21, 2026
Merged

feat(app): handle rate limits#6790
Prospector merged 5 commits into
mainfrom
fetch/app-rate-limit-handling

Conversation

@fetchfern

@fetchfern fetchfern commented Jul 21, 2026

Copy link
Copy Markdown
Member
  • Adds handling of 429 errors
  • Adds stricter local rate limiting of 120 RPM + 50 burst

Closes DEV-1239

@modrinth-bot

modrinth-bot commented Jul 21, 2026

Copy link
Copy Markdown
Member

Note

This changelog has been baked. Any further edits will not be reflected.

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

  • Fixed ratelimits not being handled properly.

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@fetchfern
fetchfern requested a review from aecsocket July 21, 2026 15:36
@Prospector
Prospector added this pull request to the merge queue Jul 21, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 21, 2026
@Prospector
Prospector added this pull request to the merge queue Jul 21, 2026
@Prospector
Prospector removed this pull request from the merge queue due to a manual request Jul 21, 2026
@Prospector
Prospector merged commit 79013c2 into main Jul 21, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants