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

Olm sessions broken by use of outdated cache #7790

Open
richvdh opened this issue May 14, 2024 · 5 comments
Open

Olm sessions broken by use of outdated cache #7790

richvdh opened this issue May 14, 2024 · 5 comments
Assignees

Comments

@richvdh
Copy link
Member

richvdh commented May 14, 2024

We recently found and fixed a bug in Element X iOS (matrix-org/matrix-rust-sdk#3110) which caused Olm sessions to be corrupted due to lack of cache invalidation when control switched between the main process and the NSE process.

According to @BillCarsonFr, Element iOS suffers the same problem; we should fix this as it causes users on other platforms to see UTD errors.

@richvdh
Copy link
Member Author

richvdh commented May 14, 2024

Estimated as "S" assuming we just delete the crypto store cache when we come to the foreground

@BillCarsonFr BillCarsonFr self-assigned this May 24, 2024
@BillCarsonFr
Copy link
Member

In RC #7798

@richvdh
Copy link
Member Author

richvdh commented Jul 24, 2024

According to @BillCarsonFr, this is still a potential problem.

The current workaround on classic ios is that we "refresh" the olm machine when the app enters foreground
So the workaround limits the possibilities of outdated cache but not 100% of them

@richvdh
Copy link
Member Author

richvdh commented Jul 25, 2024

Related: #7751

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

No branches or pull requests

2 participants