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

Recent history doesn't decrypt on login #3045

Closed
ara4n opened this issue Jul 16, 2024 · 2 comments
Closed

Recent history doesn't decrypt on login #3045

ara4n opened this issue Jul 16, 2024 · 2 comments
Labels
A-E2EE Encryption O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue

Comments

@ara4n
Copy link
Member

ara4n commented Jul 16, 2024

Steps to reproduce

  1. Login to an OIDC account (don't use QR login)
  2. Verify the login from EW to get E2EE working
  3. Open an E2EE room and discover it full of UTDs for the recent history it's synced
  4. Scroll back a few pages and discover that history then starts working.
  5. Later on, roughly 10 minutes later, notice that the recent history is now decrypted too.

Outcome

What did you expect?

Recent history is decryptable as soon as you've verified your new login.

What happened instead?

It feels like the SDK is caching the fact that it couldn't decrypt the history after the initial sync, and not retrying to decrypt once E2EE is verified.

Your phone model

iPhone 12 Pro Max

Operating system version

No response

Application version

1.6.13

Homeserver

element.io

Will you send logs?

Yes
@ara4n ara4n added the T-Defect label Jul 16, 2024
@Velin92 Velin92 added X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue S-Minor Impairs non-critical functionality or suitable workarounds exist O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-E2EE Encryption labels Jul 16, 2024
@ara4n
Copy link
Member Author

ara4n commented Jul 16, 2024

should this be S-Major, given how problematic floods of UTDs are?

@pixlwave pixlwave mentioned this issue Jul 17, 2024
Closed
@ara4n ara4n mentioned this issue Jul 23, 2024
Open
@richvdh richvdh changed the title Recent history doesn't decrypt on login on OIDC Recent history doesn't decrypt on login Jul 26, 2024
@richvdh
Copy link
Member

richvdh commented Jul 26, 2024

Let's close this in favour of #3055, which seems to be the same thing but slightly better described.

@richvdh richvdh closed this as completed Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-E2EE Encryption O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ara4n @richvdh @Velin92