Skip to content

fix(voice): cap reconnect backoff at maximumConnectDelay - #586

Merged
topi314 merged 1 commit into
disgoorg:masterfrom
thomas-vilte:fix/voice-reconnect-backoff-growth
Aug 10, 2026
Merged

fix(voice): cap reconnect backoff at maximumConnectDelay#586
topi314 merged 1 commit into
disgoorg:masterfrom
thomas-vilte:fix/voice-reconnect-backoff-growth

Conversation

@thomas-vilte

Copy link
Copy Markdown
Contributor

doReconnect's timer is time.Duration(try) * delay instead of just delay. Since try never resets, the actual wait between retries grows linearly forever instead of capping at maximumConnectDelay (10s) — even though the comment above it says "Exponentially backoff up to a limit of 10s".

Confirmed against production logs: by try ~200+, retries were landing ~2000s+ apart (try * 10s), instead of holding steady at 10s.

Fix: use delay directly for the timer.

@topi314
topi314 merged commit dcfcfad into disgoorg:master Aug 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants