Skip to content

Fix WebAuthn hints to pass on public key credential options#11050

Merged
aduth merged 1 commit intomainfrom
aduth-fix-webauthn-hints
Aug 7, 2024
Merged

Fix WebAuthn hints to pass on public key credential options#11050
aduth merged 1 commit intomainfrom
aduth-fix-webauthn-hints

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Aug 7, 2024

🛠 Summary of changes

Updates WebAuthn enrollment to pass the hints property on the correct nesting level of the configuration.

Previously, it was passed as a property of authenticatorSelection, but it is a sibling option of authenticatorSelection.

Specification: https://w3c.github.io/webauthn/#sctn-parseCreationOptionsFromJSON

Related Slack discussion: https://gsa-tts.slack.com/archives/C01710KMYUB/p1723054132991659

📜 Testing Plan

If you have Chrome 128 available (currently beta, stable release planned for 8/14/24), you can verify that you are prompted for a security key when choosing to add a security key to your account. Previously, you would see a QR code prompting for a passkey, with small text mentioning the ability to use a security key.

👀 Screenshots

Before After
image image

changelog: Bug Fixes, WebAuthn, Fix WebAuthn hints to pass on public key credential options
@aduth aduth requested a review from a team August 7, 2024 18:24
@aduth aduth merged commit 09720cb into main Aug 7, 2024
@aduth aduth deleted the aduth-fix-webauthn-hints branch August 7, 2024 19:26
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