Skip to content

Fix / Reconnection fixes - #33

Merged
quiethrs merged 3 commits into
mainfrom
fix/reconnection-fixes
Aug 20, 2026
Merged

Fix / Reconnection fixes#33
quiethrs merged 3 commits into
mainfrom
fix/reconnection-fixes

Conversation

@dyachenko-yaroslav

@dyachenko-yaroslav dyachenko-yaroslav commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

What

  • Raise Socket.IO reconnection attempts from 5 to 15, and read the count from one constant that the give-up message shares.

Why

Giving up disconnects the machine until someone restarts the app by hand. Five attempts against a delay capped at 5s ran out after roughly twenty seconds — less than a rolling deploy of the Gateway takes. Fifteen buys about a minute.

The count also appeared as a literal in the give-up message, where it would have drifted from the setting on the next change.

@dyachenko-yaroslav dyachenko-yaroslav self-assigned this Aug 5, 2026
Giving up disconnects the machine until someone restarts the app by hand, and
five attempts against a delay capped at 5s ran out after roughly twenty seconds
— less than a rolling deploy of the Gateway takes. Fifteen buys about a minute.

The attempt count also appeared as a literal in the give-up message, where it
would have drifted from the setting on the next change; both now read the same
constant.
@dyachenko-yaroslav dyachenko-yaroslav changed the title Fix / Reconnection fixes EDIT Fix / Reconnection fixes Aug 5, 2026
@dyachenko-yaroslav dyachenko-yaroslav changed the title EDIT Fix / Reconnection fixes [WIP] Fix / Reconnection fixes Aug 5, 2026
@dyachenko-yaroslav dyachenko-yaroslav changed the title [WIP] Fix / Reconnection fixes Fix / Reconnection fixes Aug 19, 2026
@quiethrs
quiethrs merged commit 1d938e4 into main Aug 20, 2026
4 checks passed
@quiethrs
quiethrs deleted the fix/reconnection-fixes branch August 20, 2026 07:00
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.

3 participants