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

server/require-user-verification #181

Merged
merged 1 commit into from
Mar 9, 2022

Conversation

MasterKale
Copy link
Owner

While dogfooding the server library I confused myself by not having a requireUserVerification option on verifyAuthenticationResponse() like there is on verifyRegistrationResponse(). Instead was the fidoUserVerification argument that felt clunky to use and actually ignored up if set to "preferred".

I remembered I'd added fidoUserVerification to satisfy a few tests in FIDO Conformance testing that require you to pass verification even when user presence was false or not set:

Screen Shot 2022-02-15 at 7 10 54 PM

However, digging into why FIDO Conformance required servers to support up not being true when the latest spec requires up to always be true, it seems in 2019 when WebAuthn was in a very early state there was some discussion about the possibility of supporting "silent authentication". The spec never adopted the idea and now, as of today, user presence must be true.

I'm attempting to reopen an issue in the FIDO Conformance Tools repo to get them to reconsider these three tests as I don't think they're relevant anymore given the evolution of WebAuthn over the last three years.

In the meantime I've prepped this diff to make verifyAuthenticationResponse() simpler to use by matching how verifyRegistrationResponse() allows you to require the uv flag be true.

@MasterKale MasterKale added this to the v5.0.0 milestone Mar 9, 2022
@MasterKale MasterKale added the package:server @simplewebauthn/server label Mar 9, 2022
@MasterKale MasterKale merged commit 1d93217 into master Mar 9, 2022
@MasterKale MasterKale deleted the server/require-user-verification branch March 9, 2022 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:server @simplewebauthn/server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant