Skip to content

Commit

Permalink
Don't zero platform-authenticator AAGUIDs.
Browse files Browse the repository at this point in the history
As discussed at the face-to-face, this reflects current practice where
the AAGUID of platform authenticators are passed through even when
attestation is not requested.
  • Loading branch information
Adam Langley committed Apr 20, 2024
1 parent 3ad0eda commit 8f87400
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions index.bs
Original file line number Diff line number Diff line change
Expand Up @@ -2136,9 +2136,9 @@ a numbered step. If outdented, it (today) is rendered as a bullet in the midst o
:: Replace potentially uniquely identifying information with non-identifying versions of the
same:
1. If the [=authData/attestedCredentialData/aaguid=] in the [=attested credential data=] is 16 zero bytes, <code>|credentialCreationData|.[=attestationObjectResult=].fmt</code> is "packed", and "x5c" is absent from <code>|credentialCreationData|.[=attestationObjectResult=]</code>, then [=self attestation=] is being used and no further action is needed.
1. Otherwise
1. Replace the [=authData/attestedCredentialData/aaguid=] in the [=attested credential data=] with 16 zero bytes.
1. Otherwise:
1. Set the value of <code>|credentialCreationData|.[=attestationObjectResult=].fmt</code> to "none", and set the value of <code>|credentialCreationData|.[=attestationObjectResult=].attStmt</code> to be an empty [=CBOR=] map. (See [[#sctn-none-attestation]] and [[#sctn-generating-an-attestation-object]]).
1. If |authenticator| is not a [=platform authenticator=] then replace the [=authData/attestedCredentialData/aaguid=] in the [=attested credential data=] with 16 zero bytes.

: {{AttestationConveyancePreference/indirect}}
:: The client MAY replace the [=authData/attestedCredentialData/aaguid=] and [=attestation statement=] with a more privacy-friendly
Expand Down

0 comments on commit 8f87400

Please sign in to comment.