Skip to content

Commit

Permalink
Merge pull request #2216 from w3c/issue-2210-compound-attStmtTemplate
Browse files Browse the repository at this point in the history
Fix CDDL inconsistencies in attStmtType and compound format
  • Loading branch information
agl authored Dec 11, 2024
2 parents 81077df + 8b29bec commit 3bc8301
Showing 1 changed file with 17 additions and 13 deletions.
30 changes: 17 additions & 13 deletions index.bs
Original file line number Diff line number Diff line change
Expand Up @@ -5737,7 +5737,7 @@ template:

- <strong>Syntax:</strong>
The syntax of an [=attestation statement=] produced in this format, defined using CDDL [[!RFC8610]] for the extension point
`$attStmtFormat` defined in [[#sctn-generating-an-attestation-object]].
`$$attStmtType` defined in [[#sctn-generating-an-attestation-object]].

- <dfn>Signing procedure</dfn>:
The [=signing procedure=] for computing an [=attestation statement=] in this [=attestation statement format|format=] given
Expand Down Expand Up @@ -5833,17 +5833,21 @@ the [=authenticator=] MUST:

```
attObj = {
authData: bytes,
$$attStmtType
}

attStmtTemplate = (
fmt: text,
attStmt: { * tstr => any } ; Map is filled in by each concrete attStmtType
)

; Every attestation statement format must have the above fields
attStmtTemplate .within $$attStmtType
authData: bytes,

; Each choice in $$attStmtType defines the fmt value and attStmt structure
$$attStmtType
} .within attStmtTemplate

attStmtTemplate = {
authData: bytes,
fmt: text,
attStmt: (
{ * tstr => any } ; Map is filled in by each concrete attStmtType
//
[ * any ] ; attStmt may also be an array
)
}
```

### Signature Formats for Packed Attestation, FIDO U2F Attestation, and Assertion Signatures ### {#sctn-signature-attestation-types}
Expand Down Expand Up @@ -6930,7 +6934,7 @@ The "compound" attestation statement format is used to pass multiple, self-conta
attStmt: [2* nonCompoundAttStmt]
)

nonCompoundAttStmt = { $$attStmtType } .within { fmt: text .ne "compound" }
nonCompoundAttStmt = { $$attStmtType } .within { fmt: text .ne "compound", * any => any }
```

: Signing procedure
Expand Down

0 comments on commit 3bc8301

Please sign in to comment.