-
Notifications
You must be signed in to change notification settings - Fork 356
Wait for the verification state to arrive after creating an account #5850
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
base: develop
Are you sure you want to change the base?
Conversation
|
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #5850 +/- ##
===========================================
- Coverage 81.43% 81.41% -0.02%
===========================================
Files 2475 2475
Lines 66127 66142 +15
Branches 8419 8422 +3
===========================================
- Hits 53852 53851 -1
- Misses 9174 9190 +16
Partials 3101 3101 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
bmarty
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks correct but when I tried to create an account, I observed the issue: the screen "Confirm your identity" was displayed.
The logs are here if it helps: https://github.com/element-hq/element-x-android-rageshakes/issues/8934
Sorry to ask, but are you sure you tested the right branch? I see in the logs a behaviour that should display a log line that isn't there, which is weird. Maybe I can upload a version with more logs and you can re-test it so we can get more info? |
…the user starts a new login/registration attempt from the onboarding screen
…he `AuthScope` so their state is provided and reset also for each attempt
…an provide it later to `MatrixAuthenticationService.loginWithOidc` Then, when an account creation is detected, wait until the verification state is updated to `VERIFIED` or a timeout happens. Move the existing login in `CreateAccountPresenter.importSession` to `MatrixAuthenticationService` too so this check is more centralised
…account creation Do the same for other `Login*` classes that handle both login and account creation
351d1a1 to
105ac7b
Compare
|
|
I my commit is 351d1a1 according to this log: https://rageshakes.element.io/api/listing/2025-12-08/134651-22LSU56H/logs.2025-12-08-13.log.gz and the app I still have on my phone, and I think I downloaded the app from Diawi. Maybe the link was outdated... I'll test again |
|
So tested again after building the app locally and creating a new account on matrix.org (
Logs: https://github.com/element-hq/element-x-android-rageshakes/issues/8968 |
|
So my use case to repro the issue was that I created the account on the MAS page, but initially in the first screen of the application I clicked on "Sign in manually" instead of "Create account". |





Content
AuthScopewith itsAuthGraphthat will be reset each time the user starts a new type of login/account creation attempt.LoginHelper.isAccountCreationtotrueso we can later provide it to theMatrixAuthenticationService.RustMatrixAuthenticationServicewill now perform a sync whenisAccountCreation == trueand wait until we receive the verification state or the attempt times out after 10 seconds.Login*classes toAuthentication*classes if they handle both logins and account creation flows.Motivation and context
Fixes #5812.
Tests
I tested this with multi-account, but it should be the same without it.
Tested devices
Checklist