-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Comments
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. |
If we're removing the "upgrade" flow, we should also remove the behaviour in |
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 doesn't work in rust crypto, and we have no plans to fix it (element-hq/element-web#27455)
it doesn't work in rust crypto, and we have no plans to fix it (element-hq/element-web#27455)
@richvdh So what do we need to do "If we detect that you have key backup, but no 4S" instead of showing this broken flow? |
Treat it the same as having no 4S (and no key backup), imho |
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:
Error bootstrapping secret storage createSecretStorageKey() callback did not return a secret storage key
which then produces obscure UI feedback like "Unable to set up secret storage" (Unable to setup secret storage
element-desktop#1910)Frankly, I don't think this is a flow we should be supporting at all. We should remove the "upgrade your encryption" flow.
The text was updated successfully, but these errors were encountered: