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

Discrepancy between example code and docs #548

Closed
RicardoE105 opened this issue Apr 4, 2024 · 4 comments
Closed

Discrepancy between example code and docs #548

RicardoE105 opened this issue Apr 4, 2024 · 4 comments

Comments

@RicardoE105
Copy link

RicardoE105 commented Apr 4, 2024

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
Copy link

majdslmt commented Apr 4, 2024

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
Copy link
Owner

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
Copy link
Owner

@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
Copy link
Owner

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

@MasterKale MasterKale closed this as not planned Won't fix, can't repro, duplicate, stale Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants