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

Wrong navigation after login #3898

Closed
bmarty opened this issue Aug 25, 2021 · 4 comments · Fixed by #4115
Closed

Wrong navigation after login #3898

bmarty opened this issue Aug 25, 2021 · 4 comments · Fixed by #4115
Assignees
Labels
T-Defect Something isn't working: bugs, crashes, hangs and other reported problems X-Regression

Comments

@bmarty
Copy link
Member

bmarty commented Aug 25, 2021

Steps to reproduce

  • Login to an existing account
  • Click on the banner to verify the session
  • Click on "Use a Recovery Passphrase or Key

What did you expect?

I should see the passphrase form first, which include an option to open the Recovery Key form.

What happened?

Recovery Key form is displayed first and if I press back I see the passphrase form

Your phone model

Samsung S9

Operating system version

Android 10

Application version and app store

1.2.0

Have you submitted a rageshake?

No

@bmarty bmarty added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Aug 25, 2021
@bmarty bmarty added this to the Sprint - Element 1.1.15 milestone Aug 25, 2021
@bmarty
Copy link
Member Author

bmarty commented Aug 25, 2021

Not reproduced with every accounts...
Can be reproduce with @benoit0816:matrix.org if necessary

@ouchadam
Copy link
Contributor

the flow occurs when we enter the no passphrase branch of https://github.com/vector-im/element-android/blob/develop/vector/src/main/java/im/vector/app/features/crypto/quads/SharedSecureStorageViewModel.kt#L108

which I assume happens when SharedSecretStorageService.getDefaultKey returns a KeyInfo with hasPassphrase=false

handleBack always assumes we've come from the passphrase step https://github.com/vector-im/element-android/blob/develop/vector/src/main/java/im/vector/app/features/crypto/quads/SharedSecureStorageViewModel.kt#L170

@bmarty if your account doesn't have a passphrase then this might be a simple case of updating the back flow to skip the passphrase if we don't have one

however if this account/device does have a passphrase then there's something else going on... 🤔

@bmarty
Copy link
Member Author

bmarty commented Sep 28, 2021

Thanks for your investigation.
I confirm your analysis using my account @benoit0816:matrix.org: it doesn't have a passphrase then this might be a simple case of updating the back flow to skip the passphrase if we don't have one

@bmarty
Copy link
Member Author

bmarty commented Sep 29, 2021

Fixed in 1.3.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Defect Something isn't working: bugs, crashes, hangs and other reported problems X-Regression
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants