-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Handle SecureKeychain failed access while passcode enabled #4625
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
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.
LGTM
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.
I am done testing but I have a question, should the remember me toggle be disabled when I:
enable passcode login > kill the app> relaunch it > instead of logging in via the passcode > tap on the cancel button?
If I were to repeat the above with faceID/fingerprint biometrics, the remember me (it actually says unlock with faceID/fingerprint) toggle is enabled.
This happens on both ios and android. Here is a recording on ios
I get [Error: User canceled the operation.] I know it is a harmless error but can we fix so it doesn’t pollute sentry?
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.
We discussed my question in a slack dm. It is a non-issue and the ticket can be moved forward. This is 🌮 🌮🌮🌮
Description
Fixed an issue that leads the application to an unusable state if the user cancels the passcode authentication, and implements error handling when we try to access SecureKeychain.
Screenshots/Recordings
RPReplay_Final1657010928.mov
Issue
Progresses #260
Checklist