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

Rejecting biometric unlock when restoring wallet still prompts for biometric unlock when trying to unlock wallet #29490

Closed
srirambv opened this issue Apr 4, 2023 · 3 comments · Fixed by brave/brave-core#17911
Assignees
Labels
bug feature/web3/wallet Integrating Ethereum+ wallet support OS/Android Fixes related to Android browser functionality QA Pass - Android ARM QA/Yes release-notes/exclude

Comments

@srirambv
Copy link
Contributor

srirambv commented Apr 4, 2023

Description

Rejecting biometric unlock when restoring wallet still prompts for biometric unlock when trying to unlock wallet

Steps to reproduce

  1. Install 1.52.15
  2. Set up (create/restore) wallet with bio-metric
  3. Restore wallet via seed phrase
  4. Cancel biometric unlock feature when prompted
  5. Lock wallet manually or let it auto-lock
  6. Prompts for biometric unlock

Actual result

Rejecting biometric unlock when restoring wallet still prompts for biometric unlock when trying to unlock wallet

Expected result

Should disable biometric unlock if user cancels it when restoring wallet

Issue reproduces how often

Easy

Version/Channel Information:

  • Can you reproduce this issue with the current Play Store version? NA
  • Can you reproduce this issue with the current Play Store Beta version? NA
  • Can you reproduce this issue with the current Play Store Nightly version? Yes

Device details

  • Install type (ARM, x86): ARM
  • Device type (Phone, Tablet, Phablet): Phone
  • Android version: 13

Brave version

1.52.15

Website problems only

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards?NA
  • Is the issue reproducible on the latest version of Chrome? NA

Additional information

cc: @Pavneet-Sing @jamesmudgett @simoarpe
@srirambv srirambv added bug QA/Yes release-notes/exclude feature/web3/wallet Integrating Ethereum+ wallet support OS/Android Fixes related to Android browser functionality labels Apr 4, 2023
@srirambv srirambv added this to Web3 Apr 4, 2023
@Pavneet-Sing
Copy link

Pavneet-Sing commented Apr 4, 2023

I am not able to reproduce this. I cancelled the bio-metric (on the black screen) and didn't get any prompt afterwards.

biometric-login-issue.webm

Update: Found the issue, The bio-metric needs to be set-up before the next restore (without wallet reset). So the expected behaviour is to clear the previous bio-metric setup if was cancelled during restore, without wallet reset. Updated the steps.

cc: @srirambv

@kjozwiak
Copy link
Member

The above requires 1.50.117 or higher for 1.50.x verification 👍

@srirambv
Copy link
Contributor Author

Verification passed on Oppo Reno 5 with Android 13 running 1.50.120 x64 build

  • Verified cancelling Biometric unlock during wallet recovery, with an existing wallet with Biometric unlock enabled, doesn't show the Biometric unlock. Additional verification done #20807 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/web3/wallet Integrating Ethereum+ wallet support OS/Android Fixes related to Android browser functionality QA Pass - Android ARM QA/Yes release-notes/exclude
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants