Skip to content

Debounce reconnect disconnect logging #1862

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.

Sign up for GitHub

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

Merged
juliusmarminge merged 1 commit into from
Apr 9, 2026
+123 −9

Debounce reconnect failure logging

c61ff7a
Select commit
Loading
Failed to load commit list.
Merged

Debounce reconnect disconnect logging #1862

Debounce reconnect failure logging
c61ff7a
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Approvability Check succeeded Apr 9, 2026 in 8m 25s

Approved (click on check for details)

  • Eligibility
  • Correctness

Note

More information about how approvability works can be found in our Help Center.

Details

Eligibility
This PR makes two small, self-contained improvements: deduplicating WebSocket disconnect warning logs when multiple subscriptions fail together, and properly catching errors from replay recovery to prevent unhandled promise rejections. Both changes are low-risk logging/error-handling fixes with corresponding test coverage.

Notes:
No code owners found in target merge base 70f5dfc

View more details on MacroscopeApp