-
Notifications
You must be signed in to change notification settings - Fork 119
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
Allow possibility to disable face unlock on Android #231
Comments
Hi @eblis, I'm the friendly issue checker. If you think it's fine to make an exception, just ignore this message. Thanks! |
Hi!, The only workaround would be implementaing the old FingerprintManager. While deprecated, it still is functional enough fir this purpose. Better explanation is here |
Does allowing just |
https://developer.android.com/reference/android/hardware/biometrics/BiometricManager.Authenticators#summary |
I would say that most devices implement this correctly and would be a nice addition. |
I wouldn't say that. Even Samsung likes to have non-standard implementations (but I am not aware of anything in the area of biometric authentication so far). Huawei, for example, does not display any text on the prompts, even if I write the biometric authentication from zero. I'm not against the feature either, I find the implementation okay for the most part, BUT without testing I find that more critical to take in productively when it's opt-out. Does it trows an error which needs to be handled specialy? Do the enduser just needs to reregister his fingerprint? It should also be tested against different API levels, so that there are no problems already when setting the flags. When it comes to security, it should be tested, especially if it could change the behavior. |
On the Samsung device I've seen the current implementation offers fingerprint unlock by default, and only if you want it switches to face unlock. |
It would be nice if you could select which authentication methods are allowed, at least on Android systems.
Some Android vendors allow both fingerprint and face unlock, but certain sensible applications, like Bitwarden shouldn't allow login via face, just by fingerprint.
An option in the code to select if you want strong authentication (fingerprint) or weak authentication (face) would be very nice.
This is a very requested feature in Bitwarden and as far as i can tell Bitwarden uses this plug-in for cross platform authentication.
The text was updated successfully, but these errors were encountered: