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

Refactor: PINEnter Screen #1424

Open
timbl-ont opened this issue Jan 31, 2025 · 0 comments
Open

Refactor: PINEnter Screen #1424

timbl-ont opened this issue Jan 31, 2025 · 0 comments

Comments

@timbl-ont
Copy link
Contributor

First stab at the requirements for the PINEnter refactoring - add others and provide feedback as required to this issue:

The PINEnter screen is used in multiple contexts that will need to be supported as part of the refactor:

  • User Authentication at wallet initialization
  • Enhanced or use case specific authentication
    • Change Biometrics
    • Change PIN
    • Proof of presence (future)
    • Other higher risk settings changes (TBD)
  • PIN reminder (future) - user is asked to enter the PIN when they are using biometrics to remind them of what there PIN is. The Signal app does this fairly well.
  • Wallet unlock after idle timeout
  • ???

In addition the user lockout behaviour on an incorrect PIN entry will change depending on the context. There are two paths:

  • Lock the user out of the wallet for a period of time after n failed attempts. The period of time should increase by x amount every n failures (with some max limit). All variables must be configurable and injectable. Used for wallet authentication flows.

  • Lock the user out of the specific PIN Enter context for period of time after n failed attempts. For example if the user fails to change their PIN then they would be locked out of only the PIN change functionality and not the entire wallet. The period of time should increase by x amount every n failures (with some max limit). All variables must be configurable and injectable. Used for enhanced authentication and PIN reminder. Variables may be different for each context???

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant