-
Notifications
You must be signed in to change notification settings - Fork 559
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
Issues with biometric authentication (or rather when it is disabled) #969
Comments
I'm going to ping @IndusAryan here as the one who worked on biometric auth to begin with. I personally only like it to work for default account anyway since we can't have a PIN there, I don't find it necessary for other accounts and seems a bit overkill but it's fine and at least provides a bit of extra nice security, so thanks for that... |
|
@IndusAryan also I found another a major issue I think, adding biometric_key to nonTransferableKeys wont stop it from backing up IIRC, nonTransferableKeys has no actual affect, it was supposed to be fixed with #623 but that seems dead at this point? #744 (comment) — but I guess maybe we should just apply that fix anyway if that isn't progressing anymore? |
oh, then you should pr, even if it is not dead , it will take months |
I'll PR that specific fix within the next couple days I guess. Hopefully tomorrow. |
Steps to reproduce
When I turn off biometric authentication I still get this which is meaningless if it is being turned off:
I see this everytime I open the app, even if it is not enabled at all in CloudStream:
Also this should not say "fingerprint" as there are other types of biometric authentication.
It also shows the "Logged in with" toast before you actually complete login with biometric if enabled, it should wait to show that for the login to actually be completed.
Expected behavior
...
Actual behavior
...
Cloudstream version and commit hash
4.3.1 bd69054
Android version
Android 14
Logcat
No response
Other details
No response
Acknowledgements
The text was updated successfully, but these errors were encountered: