-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fingerprint unlock with fingerprint unlocking disabled! #985
Comments
Hey there. thanks for reporting. I can confirm that this is definitely a bug on our end and it's not intended to work that way. The first problem is related to #927 where we are enabling it by default during sync without actually asking the user if they want to do it. The second problem is that the option "Sign in with Fingerprint" it's not reflecting the ON state after the sync. Both problems will get fixed on our next release. |
When is this expected to land in a release? |
Monday / tuesday the latest. |
Two months now. Still having this problem... Funny thing it used to work fine on previous releases. |
Hi @jumpnkick thanks for reporting, I've been trying to reproduce in a Samsung S9 but I can't. Which device are you using? Also, metamask version? Thanks! |
OnePlus 3t. |
Just tried it again. It started working now. |
--> Notice that you are prompted for a fingerprint to unlock the app!
This is a pretty big security issue for me as biometrics alone are NOT secure. While I have them setup for unlocking my phone, all of my financial apps are secured behind something I know, not something I have since getting into my phone in the first place requires something I have (biometrics + device).
The text was updated successfully, but these errors were encountered: