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

Cannot log out (stuck on key backup dialog) #19535

Open
kittykat opened this issue Oct 28, 2021 · 4 comments
Open

Cannot log out (stuck on key backup dialog) #19535

kittykat opened this issue Oct 28, 2021 · 4 comments
Labels
A-E2EE-Key-Backup O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Regression Z-Community-Testing

Comments

@kittykat
Copy link
Contributor

kittykat commented Oct 28, 2021

Steps to reproduce

A preliminary step may be to start a new DM conversation with someone new.

  1. Start logging out of an account
  2. See the dialog warning me "You'll lose access to your encrypted messages"
  3. Click the right button to "Connect this session to Key Backup" (I'm not sure what that actually means)
  4. See spinner and then same dialog again
  5. Click "Connect this session to Key Backup"
  6. See "Keys restored" dialog (confusing as I was backing up keys, not restoring them)
  7. Click "OK"
  8. Dialog closes but I'm still logged in

Outcome

What did you expect?

To be logged out

What happened instead?

I can't log out unless I press a scary button saying that I will lose my keys

URL for webapp

staging.element.io

Application version

Element version: 1.9.3
Olm version: 3.2.3

Will you send logs?

Yes

@kittykat kittykat added T-Defect S-Major Severely degrades major functionality or product features, with no satisfactory workaround A-E2EE-Key-Backup O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Oct 28, 2021
@kittykat kittykat changed the title Key backup dialog needs to be confirmed twice before it goes away Cannot log out of staging.element.io (stuck on key backup dialog) Oct 28, 2021
@dbkr
Copy link
Member

dbkr commented Oct 29, 2021

Seems it's got itself into a state where key backup isn't active, then it finds a key backup on the server, restores it to connect to it (hence asking for the password) but then doesn't go back to logging out after backing up.

@richvdh richvdh changed the title Cannot log out of staging.element.io (stuck on key backup dialog) Cannot log out (stuck on key backup dialog) Mar 15, 2023
@richvdh
Copy link
Member

richvdh commented Mar 15, 2023

A preliminary step may be to start a new DM conversation with someone new.

... or just to send an encrypted message.

Note that to reproduce this it is also necessary to start from a session that is not already connected to key backup.

1. Start logging out of an account
2. See the dialog warning me "You'll lose access to your encrypted messages"
3. Click the right button to "Connect this session to Key Backup" (I'm not sure what that actually means)
4. See spinner and then same dialog again

When I get to this point, I instead see this dialog:

image

... followed by "Backup could not be decrypted with this Security Key: please verify that you entered the correct Security Key."

... which I think is due to #23929.

6. See "Keys restored" dialog (confusing as I was backing up keys, not restoring them)

... so I never get to this stage.

Anyway we probably need to revisit this once #23929 is fixed.

@richvdh
Copy link
Member

richvdh commented Mar 15, 2023

I don't think this is Major, because a second attempt to log out (or clicking through the "I don't want my keys" dialog) works just fine.

@richvdh richvdh added S-Minor Impairs non-critical functionality or suitable workarounds exist O-Occasional Affects or can be seen by some users regularly or most users rarely and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Mar 15, 2023
@richvdh
Copy link
Member

richvdh commented Mar 15, 2023

also, it should only affect users who did not create a security phrase (and hence connect to key backup on login), which should be (I hope) a minority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE-Key-Backup O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Regression Z-Community-Testing
Projects
None yet
Development

No branches or pull requests

3 participants