Skip to content

docs: add EdDSA/Ed25519 to allowed WebAuthn signature schemes#250

Draft
mraszyk wants to merge 1 commit into
mainfrom
sea-snake/spec-ed25519-webauthn
Draft

docs: add EdDSA/Ed25519 to allowed WebAuthn signature schemes#250
mraszyk wants to merge 1 commit into
mainfrom
sea-snake/spec-ed25519-webauthn

Conversation

@mraszyk
Copy link
Copy Markdown

@mraszyk mraszyk commented May 13, 2026

Summary

Update the IC interface specification to allow EdDSA on curve Ed25519 as a third WebAuthn signature scheme (alongside ECDSA P-256 and RSA PKCS#1v1.5), mirroring the implementation change in dfinity/ic#10081.

Changes

  • docs/references/ic-interface-spec.md §Web Authentication: add EdDSA on curve Ed25519 to the allowed-schemes list; clarify the signature encoding note to call out that Ed25519 WebAuthn signatures are the raw 64-byte R || s concatenation from RFC 8032 §5.1.6 (not DER-wrapped, which only applies to ECDSA).
  • docs/references/_attachments/interface-spec-changelog.md: new entry. The version number (0.61.0) is a placeholder — happy to adjust to whatever the maintainers prefer.

Context

Was triggered by the implementation work in dfinity/ic#10081 to accept Ed25519 WebAuthn keys (e.g. NitroKey 3A authenticators emit kty=OKP / alg=EdDSA / crv=Ed25519). Reviewer @eichhorl asked whether there's a corresponding spec PR — this is it.

@mraszyk mraszyk requested review from a team as code owners May 13, 2026 11:26
@github-actions github-actions Bot added the interface-spec Changes to the IC interface specification label May 13, 2026
@mraszyk mraszyk requested a review from eichhorl May 13, 2026 11:27
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 13, 2026

🤖 Here's your preview: https://z4vwg-raaaa-aaaam-aiiga-cai.icp0.io

Copy link
Copy Markdown

@eichhorl eichhorl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, is there a changelog we should extend? The PR descriptions mentions one

@marc0olo
Copy link
Copy Markdown
Member

marc0olo commented May 13, 2026

changelog lives here: https://github.com/dfinity/developer-docs/blob/main/docs/references/ic-interface-spec/changelog.md

note: the original PR also didn't include any entry in the changelog

@mraszyk
Copy link
Copy Markdown
Author

mraszyk commented May 13, 2026

LGTM, is there a changelog we should extend? The PR descriptions mentions one

We should only update the changelog and merge this PR when it is actually rolled out in production.

@marc0olo marc0olo marked this pull request as draft May 13, 2026 14:09
@marc0olo
Copy link
Copy Markdown
Member

converted into draft to avoid accidental merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

interface-spec Changes to the IC interface specification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants