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

"Upgrade your encryption" flow is broken and should be removed #27455

Open
richvdh opened this issue May 9, 2024 · 3 comments
Open

"Upgrade your encryption" flow is broken and should be removed #27455

richvdh opened this issue May 9, 2024 · 3 comments
Labels
A-E2EE A-E2EE-Key-Backup A-Element-R Issues affecting the port of Element's crypto layer to Rust 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 T-Defect

Comments

@richvdh
Copy link
Member

richvdh commented May 9, 2024

If we detect that you have key backup, but no 4S, we pop up a toast prompting you to "Upgrade your encryption".

This is broken in several ways:

Frankly, I don't think this is a flow we should be supporting at all. We should remove the "upgrade your encryption" flow. However, if it's something that we want to keep, then it needs tests and we need to fix the bugs above.

@richvdh richvdh added T-Defect S-Major Severely degrades major functionality or product features, with no satisfactory workaround A-E2EE-Key-Backup O-Occasional Affects or can be seen by some users regularly or most users rarely A-Element-R Issues affecting the port of Element's crypto layer to Rust labels May 9, 2024
@dosubot dosubot bot added the A-E2EE label May 9, 2024
@richvdh richvdh changed the title Element-R: "Upgrade your encryption" flow is broken Element-R: Remove "Upgrade your encryption" flow May 13, 2024
@richvdh
Copy link
Member Author

richvdh commented May 13, 2024

After discussion with the team today: we're agreed that this flow should be removed altogether. When logging into an account that has key backup but the keys for that backup are not available in 4S, the only option will be to create a new key backup.

@richvdh
Copy link
Member Author

richvdh commented Jun 11, 2024

If we're removing the "upgrade" flow, we should also remove the behaviour in RestoreKeyBackupDialog which prompts you to enter your backup key directly (referring to it, confusingly, as "security key"). Related: #27469

@crjo
Copy link

crjo commented Jun 18, 2024

Hi there, every new user on our on prem instance is unable to have a correct key backup at first login, is there a workaround ?
It's not occasionally for us, it's every time.

richvdh added a commit to matrix-org/matrix-js-sdk that referenced this issue Jul 11, 2024
it doesn't work in rust crypto, and we have no plans to fix it
(element-hq/element-web#27455)
github-merge-queue bot pushed a commit to matrix-org/matrix-js-sdk that referenced this issue Jul 13, 2024
it doesn't work in rust crypto, and we have no plans to fix it
(element-hq/element-web#27455)
@richvdh richvdh changed the title Element-R: Remove "Upgrade your encryption" flow "Upgrade your encryption" flow is broken and should be removed Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE A-E2EE-Key-Backup A-Element-R Issues affecting the port of Element's crypto layer to Rust 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 T-Defect
Projects
None yet
Development

No branches or pull requests

2 participants