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

fix MFA login with security key #1968

Merged
merged 2 commits into from
Nov 7, 2024
Merged

fix MFA login with security key #1968

merged 2 commits into from
Nov 7, 2024

Conversation

FreddyDevelop
Copy link
Contributor

Description

The MFA login with security key due to wrongly used validation method of the webauthn library. The security key validation must not use the validation method for discoverable credentials, because the MFA credential is not created as a discoverable credential. Because of the usage of the wrong method a MFA login with a security key never succeeds.

Implementation

Use the correct webauthn validation method to validate the MFA security key response.

Tests

  1. Create a new user with a security key as MFA method
  2. Try to login with the security key

The MFA login with security key due to wrongly used validation method of the webauthn library. The security key validation must not use the validation method for discoverable credentials, because the MFA credential is not created as a discoverable credential. Because of the usage of the wrong method a MFA login with a security key never succeeds.
@FreddyDevelop FreddyDevelop changed the title fix security key login fix MFA lgin with security key Nov 7, 2024
@FreddyDevelop FreddyDevelop changed the title fix MFA lgin with security key fix MFA login with security key Nov 7, 2024
@FreddyDevelop FreddyDevelop merged commit 36e3309 into main Nov 7, 2024
8 checks passed
@FreddyDevelop FreddyDevelop deleted the fix-security-key-login branch November 7, 2024 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Recently closed
Development

Successfully merging this pull request may close these issues.

2 participants