modelcontextprotocol/access
Watch
1
0
Fork
You've already forked access
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(discord): widen rate-limit jitter to de-sync parallel retries

Browse source 
The 250ms jitter was too narrow for ~68 parallel refresh reads — all
retries clustered after the same retry_after and collided again. Scale
jitter linearly with attempt number so the herd progressively spreads
out across attempts.
This commit is contained in:
David Soria Parra 2026-03-21 21:34:26 +00:00
commit 9c7f6f3051
No known key found for this signature in database
1 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/discord.ts
View file

@ -50,7 +50,10 @@ async function discordFetch<T>(
if (response.status === 429) { if (response.status === 429) {
const body = (await response.json()) as DiscordRateLimitResponse; const body = (await response.json()) as DiscordRateLimitResponse;
const retryAfterMs = Math.ceil(body.retry_after * 1000) + Math.random() * 250; // Linearly increasing jitter de-syncs the thundering herd when many
// resources refresh in parallel and all receive the same retry_after.
const jitterMs = Math.random() * 1000 * (attempt + 1);
const retryAfterMs = Math.ceil(body.retry_after * 1000) + jitterMs;
lastError = new Error( lastError = new Error(
`Discord API rate limited on ${endpoint} (retry_after=${body.retry_after}s, global=${body.global})` `Discord API rate limited on ${endpoint} (retry_after=${body.retry_after}s, global=${body.global})`
); );