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

Proposed change to RTN20: restart in-progress connection attempts if an online event occurs in the connecting state #132

Closed
paddybyers opened this issue Feb 28, 2023 · 3 comments · Fixed by #135
Assignees

Comments

@paddybyers
Copy link
Member

See https://sdk.ably.com/builds/ably/specification/main/features/#RTN20

The aim of RTN20 is to ensure that a client reconnects as soon as possible after the host comes online.

However, it has been reported that reconnections can take a long time if the client is connecting at the time of that online event - a connection attempt was initiated whilst offline, and is blocked in some state, but doesn't succeed as a result of the host coming online - it can stay blocked until a timeout occurs, and then the client will wait the usual retry interval (30s if it was suspended) before retrying.

A way around this would be for RTN20 to require that an in-progress connection attempt is cancelled and restarted if the online event happens whilst in the connecting state.

Internal discussion: https://ably-real-time.slack.com/archives/C01EHJ8J5MG/p1676973802248769?thread_ts=1676913499.160499&cid=C01EHJ8J5MG

@sync-by-unito
Copy link

sync-by-unito bot commented Feb 28, 2023

➤ Automation for Jira commented:

The link to the corresponding Jira issue is https://ably.atlassian.net/browse/SDK-3434

@SimonWoolf
Copy link
Member

sgtm

@owenpearson owenpearson transferred this issue from ably/features Mar 2, 2023
@ably ably deleted a comment from sync-by-unito bot Mar 15, 2023
@JCB-K
Copy link

JCB-K commented Mar 20, 2023

Hi! What's the priority on this proposal? It would make a big difference in performance for an upcoming key feature of ours.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

4 participants