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

Clarify queueing behaviour for pending Presence messages #130

Open
paddybyers opened this issue Feb 1, 2023 · 1 comment
Open

Clarify queueing behaviour for pending Presence messages #130

paddybyers opened this issue Feb 1, 2023 · 1 comment

Comments

@paddybyers
Copy link
Member

The spec is ambiguous in respect of this question.

Presence messages are queued in a channel while in the ATTACHING state (and later retried following any suspend). If there is a presence update/enter already queued for a given clientId, and then there is a superseding update for the same clientId - are both supposed to be sent to Ably when the channel completes attachment, or is it acceptable only to send the later update (provided both callbacks get called)?

(Internal discussion: https://ably-real-time.slack.com/archives/CURL4U2FP/p1675263889316679.)

Both approaches appear to be acceptable in the current spec - so either we should state that explicitly, or require one behaviour or the other. FWIW ably-js sends all messages, avly-java discards superseded messages for a given clientId.

@sync-by-unito
Copy link

sync-by-unito bot commented Feb 1, 2023

➤ Automation for Jira commented:

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

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

No branches or pull requests

1 participant