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

Discrepancy between example code and docs #548

Closed RicardoE105 closed 7 months ago

RicardoE105 commented 7 months ago

Describe the issue

On the example code, specifically when verifying the authentication, the property requireUserVerification is set to true, but in the docs, requireUserVerification it's not set. I'm using a Yubi Key, and the authentication only works when requireUserVerification is true during verification. What am I missing?

majdslmt commented 7 months ago

I am trying to use this as server side with flutter app using https://pub.dev/packages/webauthn but the flutter app create payload register/verify like this {"authData":"dKbqkhPJnC90siSSsyDPQCYqlMGpUKA5fyklC2CEHvBFAAAAAAAAAAAAAAAAAAAAAAAAAAAAIApuetyDVhvSGAF7wJ_sJ6npdSb3PSuDkRUo-0Jd-taEpQECAyYgASFYIHDQhpLZ3q_4z5g6JvCdbsUFlhWRoAMmWBcBcuxVQ9JAIlggE9oA2sel6AqVSSCxl-uq62SOb64Gbc-MoUXX3cpYccE","fmt":"packed","attStmt":{"alg":-7,"sig":"MEYCIQDRxekD1zqdMn8Xu-eFSd3KIy78OJd77hR-jxeFs5304wIhAItP7JhcU36KDZ4Za7n3PrlzaYB-Iq8r-vYXfjTSMh2A"}} so i can use as a custom payload

MasterKale commented 7 months ago

I'm using a Yubi Key, and the authentication only works when requireUserVerification is true during verification. What am I missing?

Hello @RicardoE105, what error are you getting when requireUserVerification is false or omitted? I'm not sure I understand what problem you're reporting, especially since you omitted the rest of the issue template that everyone is prompted to fill out when reporting issues like this one.

MasterKale commented 7 months ago

@majdslmt if you have an issue with this library please create a separate Issue. Otherwise if your question is about using this library to verify responses generated by a different library then please start a Discussion instead.

MasterKale commented 7 months ago

@RicardoE105 feel free to reopen this issue if you believe there's still an issue here.