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

Queue and retry to-device msgs, so E2EE works in bad networks or with unreliable HS #673

Closed
4 tasks done
kittykat opened this issue Oct 3, 2022 · 4 comments
Closed
4 tasks done
Labels
A-E2EE App: Android App: iOS O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Epic Issue is at Epic level Team: Crypto Z-Chronic

Comments

@kittykat
Copy link
Contributor

kittykat commented Oct 3, 2022

Queue the to-device messages. We should make sure that we retry to-device messages. New implementation on web saves them in a queue to persistent storage to be sent when they can be.

These issues must be resolved for this issue to be considered done:

Tasks

Preview Give feedback
  1. A-E2EE O-Uncommon S-Major T-Defect Z-Chronic
    duxovni
  2. T-Enhancement
  3. A-E2EE O-Frequent S-Major T-Enhancement Team: Crypto
@kittykat kittykat added T-Epic Issue is at Epic level O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround A-E2EE Z-Chronic labels Oct 3, 2022
@dkasak
Copy link
Member

dkasak commented Oct 10, 2022

Should be noted that matrix-org/matrix-js-sdk#2549 only retries room key (Megolm session) sending; other to-device messages are still not retried. This is tracked in matrix-org/matrix-js-sdk#1866.

@jdauphant
Copy link

@giomfo maybe we can push this ticket to solve part of our keys sharing problems.

@pmaier1
Copy link
Contributor

pmaier1 commented Jan 20, 2023

According to @BillCarsonFr, this will be solved for other platforms once we have Element R integrated which will happen soon.
The web-specific part (#673 (comment)) will also be fixed by integrating the rust SDK.

@pmaier1
Copy link
Contributor

pmaier1 commented Feb 28, 2023

@BillCarsonFr Can you close this and the related tasks if they are solved by Element R? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE App: Android App: iOS O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Epic Issue is at Epic level Team: Crypto Z-Chronic
Projects
None yet
Development

No branches or pull requests

5 participants