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

Allow when user_verification is preferred and PIN is not set #148

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alexandrezia
Copy link

When user_verification is "preferred" and it's not configured in ubikey, allow authentication to proceed,
As stated here: https://developers.yubico.com/WebAuthn/WebAuthn_Developer_Guide/User_Presence_vs_User_Verification.html

PREFERRED: This value indicates that the RP prefers user verification for the operation if possible, but will not fail the operation if the response does not have the AuthenticatorDataFlags.UV flag set.

@dainnilsson
Copy link
Member

I believe the current behavior is correct: PREFERRED should be treated as REQUIRED when the Authenticator supports it (even if it isn't configured). This is in accordance with the behavior I am seeing in Windows and what I interpret from the CTAP 2 and WebAuthn specifications.

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

Successfully merging this pull request may close these issues.

None yet

3 participants