Skip to content

[PM-3393] Excessive Invalid Biometric unlock attempts should automatically log out TDE users#2747

Merged
andrebispo5 merged 9 commits intomasterfrom
auth/pm-3393/biometric-exceed
Sep 19, 2023
Merged

[PM-3393] Excessive Invalid Biometric unlock attempts should automatically log out TDE users#2747
andrebispo5 merged 9 commits intomasterfrom
auth/pm-3393/biometric-exceed

Conversation

@andrebispo5
Copy link
Contributor

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

When the user has no master password or pin and exceeds the biometric login attempts, the app should logged out.

Code changes

Add action to IsBiometricIntegrityValidAsync to invoke when the TooManyAttempts is triggered.
Add to AppHelpers a method to check if the user doesn't have other login methods available, show a warning message and logout.

Screenshots

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@andrebispo5 andrebispo5 requested a review from a team September 7, 2023 08:46
@bitwarden-bot
Copy link

bitwarden-bot commented Sep 7, 2023

Logo
Checkmarx One – Scan Summary & Details090eb84b-827f-4cb2-ae0f-18778900ef8e

No New Or Fixed Issues Found

@andrebispo5 andrebispo5 requested a review from fedemkr September 8, 2023 15:51
@andrebispo5 andrebispo5 merged commit a6f0533 into master Sep 19, 2023
@andrebispo5 andrebispo5 deleted the auth/pm-3393/biometric-exceed branch September 19, 2023 09:32
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

Successfully merging this pull request may close these issues.

3 participants