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

Users should be able to read their existing history rapidly when logging in on a new device. #757

Open
2 of 4 tasks
BillCarsonFr opened this issue Oct 14, 2022 · 1 comment
Labels
A-E2EE App: Android App: iOS App: web O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Critical Prevents work, causes data loss and/or has no workaround T-Enhancement T-Epic Issue is at Epic level Team: Crypto X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue Z-Chronic

Comments

@BillCarsonFr
Copy link
Member

BillCarsonFr commented Oct 14, 2022

Large key backups take forever to restore (or fail entirely); should restore incrementally instead

Web

  1. A-E2EE A-E2EE-Key-Backup A-Performance O-Uncommon S-Major T-Defect Z-Legacy-Crypto
  2. A-E2EE-Key-Backup O-Occasional S-Major T-Defect
  3. A-E2EE A-E2EE-Key-Backup

Rust

  1. encryption enhancement

IOS: Similar issue

Android:
Better but still some issues, incremental backup calls done after each syncs so can take some time if not a lot to sync. And it's not working that well for large gappy syncs, in this case the crypto service is catching up on a lot of to_devices that will trigger upload of the newly received key and looks like it's blocking the backup from incrementally downloading keys

How to test:

  • Login an existing account with a big key backup.
  • Open e2ee rooms, history should decrypt quickly (seconds)
@karlabbott karlabbott added O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience T-Enhancement labels Oct 26, 2022
@kittykat kittykat added the S-Critical Prevents work, causes data loss and/or has no workaround label Jan 19, 2023
@richvdh richvdh added X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue and removed rust-sdk labels Mar 15, 2023
@richvdh
Copy link
Member

richvdh commented Mar 1, 2024

requires a fix to element-hq/element-web#27009 ?

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 App: web O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Critical Prevents work, causes data loss and/or has no workaround T-Enhancement T-Epic Issue is at Epic level Team: Crypto X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue Z-Chronic
Projects
None yet
Development

No branches or pull requests

4 participants