Skip to content
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.

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

Reconnect websockets lazily on usage #222

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

boxdot
Copy link
Collaborator

@boxdot boxdot commented Dec 19, 2023

When identified or unidentified websocket is used, it is checked whether it is closed, and in that case it is reconnected. This is less powerful than an auto-reconnecting websocket, because the messages stream is interrupted. But this is a very simple implementation which works quite well on application level.

When identified or unidentified websocket is used, it is checked whether
it is closed, and in that case it is reconnected. This is less powerful
to auto-reconnecting websockets, because the messages stream would not
be interrupted. But this is a very simple implementation which works
quite well on application level.
@gferon gferon enabled auto-merge (squash) December 19, 2023 09:26
@gferon
Copy link
Collaborator

gferon commented Dec 19, 2023

Nice, thanks!

@gferon gferon merged commit 2d9a02b into whisperfish:main Dec 19, 2023
4 checks passed
@boxdot boxdot deleted the lazy-websockets-reconnect branch December 19, 2023 09:29
boxdot added a commit to boxdot/gurk-rs that referenced this pull request Dec 19, 2023
boxdot added a commit to boxdot/gurk-rs that referenced this pull request Dec 19, 2023
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.

2 participants