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

Technical review: Add WebAuthn Signal API #37557

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chrisdavidmills
Copy link
Contributor

@chrisdavidmills chrisdavidmills commented Jan 8, 2025

Description

Chrome 132 supports the WebAuthn Signal API. Specifically, this means support for the following static methods:

  • PublicKeyCredential.signalAllAcceptedCredentials()
  • PublicKeyCredential.signalCurrentUserDetails()
  • PublicKeyCredential.signalUnknownCredential()

This PR adds reference pages for all of the above methods. It also attempts to consolidate all the information on MDN about discoverable credentials into a single section on the WebAuthn home page. I noticed that quite a few details were dotted around on various pages, and wanted to link all of the reference pages I added to a single place for more information, rather than having to explain discoverable credentials on each page.

Please let me know if this writeup of discoverable credentials sounds correct.

Motivation

Additional details

Related issues and pull requests

  • BCD is already available for these methods.

@chrisdavidmills chrisdavidmills requested a review from a team as a code owner January 8, 2025 14:44
@chrisdavidmills chrisdavidmills requested review from sideshowbarker and removed request for a team January 8, 2025 14:45
@github-actions github-actions bot added Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed labels Jan 8, 2025
@chrisdavidmills chrisdavidmills changed the title Add WebAuthn Signal API Technical review: Add WebAuthn Signal API Jan 8, 2025
Copy link
Contributor

github-actions bot commented Jan 8, 2025

Preview URLs (9 pages)
Flaws (1)

Note! 8 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/API/Web_Authentication_API
Title: Web Authentication API
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/PublicKeyCredential/isConditionalMediationAvailable redirects to /en-US/docs/Web/API/PublicKeyCredential/isConditionalMediationAvailable_static
External URLs (17)

URL: /en-US/docs/Web/API/Web_Authentication_API
Title: Web Authentication API


URL: /en-US/docs/Web/API/PublicKeyCredential
Title: PublicKeyCredential


URL: /en-US/docs/Web/API/PublicKeyCredential/signalUnknownCredential_static
Title: PublicKeyCredential: signalUnknownCredential() static method


URL: /en-US/docs/Web/API/PublicKeyCredential/signalAllAcceptedCredentials_static
Title: PublicKeyCredential: signalAllAcceptedCredentials() static method


URL: /en-US/docs/Web/API/PublicKeyCredential/signalCurrentUserDetails_static
Title: PublicKeyCredential: signalCurrentUserDetails() static method

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant