- From: Adam Langley via GitHub <sysbot+gh@w3.org>
- Date: Wed, 11 Dec 2024 20:06:23 +0000
- To: public-webauthn@w3.org
agl has just merged emlun's pull request 2216 for https://github.com/w3c/webauthn:
== Fix CDDL inconsistencies in attStmtType and compound format ==
Closes #2210. Also fixes a couple of other issues I noticed while working on this:
- [ยง6.5.2. Attestation Statement Formats](https://w3c.github.io/webauthn/#sctn-attestation-formats) refers to a CDDL extension point `$attStmtFormat` that (no longer?) exists, it should be `$$attStmtType`.
- The `.within { fmt: text .ne "compound" }` control on `nonCompoundAttStmt` inadvertently makes `nonCompoundAttStmt` unable to match anything, because maps are matched exhaustively (see [CDDL](https://datatracker.ietf.org/doc/html/rfc8610#section-2.1), so the right-hand-side of `.within` forbids additional fields beyond `fmt`.
<!--
This comment and the below content is programmatically generated.
You may add a comma-separated list of anchors you'd like a
direct link to below (e.g. #idl-serializers, #idl-sequence):
Don't remove this comment or modify anything below this line.
If you don't want a preview generated for this pull request,
just replace the whole of this comment's content by "no preview"
and remove what's below.
-->
***
<a href="https://pr-preview.s3.amazonaws.com/w3c/webauthn/pull/2216.html" title="Last updated on Nov 27, 2024, 12:37 PM UTC (8b29bec)">Preview</a> | <a href="https://pr-preview.s3.amazonaws.com/w3c/webauthn/2216/3bba180...8b29bec.html" title="Last updated on Nov 27, 2024, 12:37 PM UTC (8b29bec)">Diff</a>
See https://github.com/w3c/webauthn/pull/2216
--
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Wednesday, 11 December 2024 20:06:24 UTC