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

Verification dialog is misleading if you managed to never set up SSSS #18536

Closed
dbkr opened this issue Aug 13, 2021 · 0 comments · Fixed by matrix-org/matrix-react-sdk#6811
Closed
Assignees
Labels
A-E2EE-Cross-Signing 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

Comments

@dbkr
Copy link
Member

dbkr commented Aug 13, 2021

Steps to reproduce

  1. Create a fresh account
  2. Clear cookies & log in again, or log in from a different browser/session
  3. After logging in, the verify dialog just has 'Use another login' because the passphrase option is missing (you never set one) and you may not even have another login

What happened?

This dialog also has a spinner on it which is inappropriate because we've trained everyone that spinner === app is busy, please wait, but we mean, "app is waiting for you to do something" so most people just click 'Use another login' and get confused by the fact that it sits there & does nothing (even if they do have another login).

What did you expect?

We probably need to admit to the user that their cross-signing is in a broken state and needs to be reset, and just offer them a reset button. Otherwise, do the automatic part of encryption set up at a different point so you can't get stuck in this mode.

Operating system

No response

Browser information

No response

URL for webapp

No response

@dbkr dbkr added O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist labels Aug 16, 2021
@dbkr dbkr added this to the App Team 1.8.4 milestone Sep 8, 2021
@duxovni duxovni self-assigned this Sep 17, 2021
@novocaine novocaine modified the milestones: App Team 1.8.6, App Team 1.8.7 Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE-Cross-Signing 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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants