Skip to content

Commit

Permalink
Added explanation...
Browse files Browse the repository at this point in the history
...as to what this new field is for.
  • Loading branch information
balfanz committed Nov 9, 2017
1 parent fda8622 commit 59683f6
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions index.bs
Original file line number Diff line number Diff line change
Expand Up @@ -730,7 +730,7 @@ When this method is invoked, the user agent MUST execute the following algorithm

1. Let |collectedClientData| be a new {{CollectedClientData}} instance whose fields are:
: {{CollectedClientData/type}}
:: The string `webauthn.create`.
:: The string "webauthn.create".
: {{CollectedClientData/challenge}}
:: The [=base64url encoding=] of |options|.{{MakePublicKeyCredentialOptions/challenge}}.
: {{CollectedClientData/origin}}
Expand Down Expand Up @@ -975,7 +975,7 @@ When this method is invoked, the user agent MUST execute the following algorithm

1. Let |collectedClientData| be a new {{CollectedClientData}} instance whose fields are:
: {{CollectedClientData/type}}
:: The string `webauthn.get`.
:: The string "webauthn.get".
: {{CollectedClientData/challenge}}
:: The [=base64url encoding=] of |options|.{{PublicKeyCredentialRequestOptions/challenge}}
: {{CollectedClientData/origin}}
Expand Down Expand Up @@ -1631,8 +1631,9 @@ following Web IDL.
</pre>

<div dfn-type="dict-member" dfn-for="CollectedClientData">
The <dfn>type</dfn> member contains the string `webauthn.create` when creating new credentials, and `webauthn.get`
when getting an assertion from an existing credential.
The <dfn>type</dfn> member contains the string "webauthn.create" when creating new credentials, and "webauthn.get"
when getting an assertion from an existing credential. The purpose of this member is to prevent certain types of signature
confusion attacks (where an attacker substitutes one legitimate signature for another).

The <dfn>challenge</dfn> member contains the base64url encoding of the challenge provided by the RP.

Expand Down

0 comments on commit 59683f6

Please sign in to comment.