Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@danpopenko
Copy link
Contributor

@danpopenko danpopenko commented Apr 29, 2025

Added extensions support for WebAuthnP256.sign, similar to WebAuthnP256.createCredential
Changed extension behavior in getCredentialCreationOptions

Why it’s important?

  • WebAuthnP256.createCredential already supports extensions
  • According to the specs, credential creation and request both support extensions

@vercel
Copy link

vercel bot commented Apr 29, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ox ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 12, 2025 2:55pm

@danpopenko danpopenko changed the title Add extensions support for WebAuthnP256.sign feat: add extensions support for WebAuthnP256.sign Apr 29, 2025
@danpopenko danpopenko requested a review from tmm May 2, 2025 09:30
@danpopenko
Copy link
Contributor Author

Thanks @tmm for the review - applied all suggestions and updated getCredentialCreationOptions in the last commit for consistency

@danpopenko
Copy link
Contributor Author

Hi @tmm! Whenever you have a moment, could you take a look at my PR? Let me know if you'd like any changes and I'll get them sorted. Thanks!

@vercel
Copy link

vercel bot commented Jun 17, 2025

@tmm is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

@tmm tmm merged commit 72209ef into wevm:main Jun 17, 2025
4 of 7 checks passed
@github-actions github-actions bot mentioned this pull request Jun 17, 2025
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

Successfully merging this pull request may close these issues.

2 participants