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

Users with no biometric features enrolled can enable biometric authentication #6713

Closed
jmartinesp opened this issue Aug 2, 2022 · 0 comments · Fixed by #6714
Closed

Users with no biometric features enrolled can enable biometric authentication #6713

jmartinesp opened this issue Aug 2, 2022 · 0 comments · Fixed by #6714
Assignees
Labels
A-Pin code T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@jmartinesp
Copy link
Member

Steps to reproduce

  1. Make sure you have no biometric authenticators enrolled in your device (no fingerprint, no face unlock, etc.).
  2. Open the app, go to Settings > Security & Privacy > Protect Access.
  3. Enable PIN code.
  4. See 'Enable biometrics' can be toggled.

Outcome

What did you expect?

'Enable biometrics' should be disabled so it can't be toggled, as there are no biometric authenticators enrolled.

What happened instead?

It is enabled.

This can lead to crashes when starting the app, since we're asking for biometric auth but there is no biometric feature to unlock the prompt, and a UserNotAuthenticatedException will be thrown.

This can also be reproduced as a weird bug in some emulators even if there are biometric features enrolled.

Your phone model

Emulator, Mi 9T Pro

Operating system version

Android 12

Application version and app store

1.4.32-dev [212085012] (G-1411fdd9) develop

Homeserver

matrix.org

Will you send logs?

No

Are you willing to provide a PR?

Yes

@jmartinesp jmartinesp added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems A-Pin code labels Aug 2, 2022
@jmartinesp jmartinesp self-assigned this Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Pin code T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
1 participant