search

MasterKale / SimpleWebAuthn

WebAuthn, Simplified. A collection of TypeScript-first libraries for simpler WebAuthn integration. Supports modern browsers, Node, Deno, and more.
https://simplewebauthn.dev
MIT License
1.62k stars 137 forks source link

RequireUserVerification Is Not Documented #564

Closed megaconfidence closed 6 months ago

megaconfidence commented 7 months ago

Describe the issue

requireUserVerification is not documented and leads to issues when using Chrome's built-in passkey because it doesn't promp for pin verification. requireUserVerification is true by default and it's behaviour should be documented as such link to implementation

MasterKale commented 7 months ago

Hello @megaconfidence, the docstring on that parameter currently notes that requireUserVerification defaults to true:

image

This was part of the v10 release because, I agree, in prior releases this property (and many others) were inadequately documented.

Does this resolve your issue?

MasterKale commented 6 months ago

Closing because the conversation has gone quiet, please feel free to reopen if you want to continue talking about this.