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

Restore from backup prompts for security key twice #23929

Closed
germain-gg opened this issue Dec 7, 2022 · 3 comments
Closed

Restore from backup prompts for security key twice #23929

germain-gg opened this issue Dec 7, 2022 · 3 comments
Labels
A-E2EE-Key-Backup O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Team: Crypto

Comments

@germain-gg
Copy link
Contributor

germain-gg commented Dec 7, 2022

Steps to reproduce

  1. Go to settings, click restore from backup
  2. Enter your security key
  3. Wait for all the keys to be fetched
  4. See you're prompted to enter your security key again
  5. Enter the same one as in step 2

Outcome

What did you expect?

To not even be asked to enter my security key, as I already did it in step 2

What happened instead?

It asked me to enter my security key, and on top of that told me there was a mismatch...
But all the messages now seem to decrypt in the background, so i don't even know what this step is doing

Operating system

No response

Application version

No response

How did you install the app?

No response

Homeserver

No response

Will you send logs?

No

@germain-gg germain-gg added T-Defect A-E2EE-Key-Backup O-Uncommon Most users are unlikely to come across this or unexpected workflow labels Dec 7, 2022
@turt2live
Copy link
Member

I suspect this is related to #23931

@turt2live turt2live added X-Needs-Info This issue is blocked awaiting information from the reporter X-Needs-Investigation Team: Crypto labels Dec 7, 2022
@kittykat kittykat added the S-Major Severely degrades major functionality or product features, with no satisfactory workaround label Dec 8, 2022
@richvdh richvdh changed the title Restore from backup security key Restore from backup prompts for security key twice Mar 15, 2023
@richvdh
Copy link
Member

richvdh commented Mar 15, 2023

I wasn't able to reproduce this exactly as phrased: whenever I click on "Restore from backup", the key is already cached and it just tells me that it successfully restored the keys.

However I was able to reproduce something very similar as follows:

  1. Start a new session
  2. Do not verify the new device or enter the security key/phrase. Ignore the dire warnings and click "I'll verify later".
  3. On the settings panel, click "Connect this session to Key Backup"
  4. Enter your security key
  5. Wait for all the keys to be fetched
  6. See you're prompted to enter your security key again [Note: in this case it prompts for your key, implying the security phrase is insufficient

It turns out that if you skip over the prompt at step 6 then everything works fine.

I'll investigate this as part of my work in wiring up Key Backup in element R (#24828)

@richvdh richvdh added S-Minor Impairs non-critical functionality or suitable workarounds exist and removed X-Needs-Info This issue is blocked awaiting information from the reporter S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Needs-Investigation labels Mar 15, 2023
@richvdh
Copy link
Member

richvdh commented Nov 7, 2023

Closing this in favour of #26247

@richvdh richvdh closed this as completed Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE-Key-Backup O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Team: Crypto
Projects
None yet
Development

No branches or pull requests

4 participants