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.

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

Jump to bottom

Workaround to fetch all the pending toDevice events from a Synapse homeserver #4612

Closed
bmarty opened this issue Dec 1, 2021 · 1 comment · Fixed by #4614
Closed

Workaround to fetch all the pending toDevice events from a Synapse homeserver #4612

bmarty opened this issue Dec 1, 2021 · 1 comment · Fixed by #4614
Assignees
@bmarty
Labels
A-Performance A-Sync O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround

Comments

@bmarty
Copy link
Member

bmarty commented Dec 1, 2021

Due to a bug on Synapse, the Android Matrix SDK can receive toDevice events with a long delay: 30s max if nothing else (new message, etc.) triggers a sync response to be sent.

The workaround implemented on some clients is to set the timeout parameter of the sync request to 0 if the previous sync response contains some toDevice events, to force Synapse to send all the pending toDevice events, if any, immediately.

Once matrix-org/synapse#11457 will be fixed, the workaround could be removed. But I think it's still worth to implement it, because once fixed it can take time for the Synapse fix to be deployed on all the Synapse instances in the wild. Also we do not have any due date for the Synapse fix.

Note: the Android Matrix SDK is managing the sync requests, so implementing this will still be useful once we will use the Crypto Rust SDK.
@bmarty bmarty added A-Performance A-Sync S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely labels Dec 1, 2021
@bmarty bmarty self-assigned this Dec 1, 2021
@bmarty bmarty mentioned this issue Dec 1, 2021
Merged
@bmarty
Copy link
Member Author

bmarty commented Dec 30, 2021

Fixed in EA 1.3.13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bmarty bmarty

Labels
A-Performance A-Sync O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround
Projects
None yet
Milestone
No milestone
1 participant
@bmarty