Skip to content

Commit

Permalink
Merge pull request #2000 from sbweeden/sweeden_1998
Browse files Browse the repository at this point in the history
Clarify validation step for packed attestation certificate for RPs.
  • Loading branch information
sbweeden committed Nov 29, 2023
2 parents f8163ea + 73eb670 commit 065b836
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions index.bs
Original file line number Diff line number Diff line change
Expand Up @@ -5890,6 +5890,10 @@ The attestation certificate MUST have the following fields/extensions:
`1.3.6.1.4.1.45724.1.1.4` (`id-fido-gen-ce-aaguid`) MUST be present, containing the AAGUID as a 16-byte OCTET STRING.
The extension MUST NOT be marked as critical.

As [=[RPS]=] may not know if the attestation root
certificate is used for multiple authenticator models, it is suggested that [=[RPS]=] check if the extension
is present, and if it is, then validate that it contains that same AAGUID as presented in the [=attestation object=].

Note that an X.509 Extension encodes the DER-encoding of the value in an OCTET STRING.
Thus, the AAGUID MUST be wrapped in <i>two</i> OCTET STRINGS to be valid. Here is a sample, encoded Extension structure:

Expand Down

0 comments on commit 065b836

Please sign in to comment.