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

authenticatorGetAssertion has no ConstraintError step for requireUserVerification #983

Closed
emlun opened this issue Jul 4, 2018 · 4 comments

Comments

@emlun
Copy link
Member

emlun commented Jul 4, 2018

authenticatorMakeCredential has the step

  1. If requireUserVerification is true and the authenticator cannot perform user verification, return an error code equivalent to "ConstraintError" and terminate the operation.

authenticatorGetAssertion also has a requireUserVerification parameter, but no equivalent step returning a "ConstraintError" if it is not supported. Step 7 reads

  1. [...]

If requireUserVerification is true, the method of obtaining user consent MUST include user verification.

[...]

but leaves unspecified what should happen if this MUST cannot be satisfied.

CTAP does return identical error codes from both operations if the argument value is unsupported, so adding the missing step to authenticatorGetAsserion would not affect compatibility with CTAP.

@nadalin
Copy link
Contributor

nadalin commented Jul 5, 2018

@emlun un you MUST, we don't have alternatives to MUSTS, suggest to close no action

@emlun
Copy link
Member Author

emlun commented Jul 6, 2018

Sorry, I don't understand what you mean by this:

un you MUST, we don't have alternatives to MUSTS

@emlun
Copy link
Member Author

emlun commented Jul 11, 2018

Decided on 2018-07-11 WG call to close this.

@emlun emlun closed this as completed Jul 11, 2018
@emlun
Copy link
Member Author

emlun commented Jul 11, 2018

This situation should not happen because the client algorithm specifies

If options.userVerification is set to required and the authenticator is not capable of performing user verification, continue.

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

No branches or pull requests

2 participants