Closed Bug 1384307 Opened 5 years ago Closed 5 years ago
Authn Public Key Credential object's "id" and "type" fields must be set
59 bytes, text/x-review-board-request
The "PublicKeyCredential" WebIDL object's "id", "discovery", and "type" fields are blank in Firefox Nightly. Both of those fields are inherited from Credential Management's WebIDL "Credential" type.  "id" should be set to the base64url encoding of the data in "rawId". "type" should be the literal "public-key" "discovery" should be the literal "remote", but as an "internal slot" it doesn't appear in the WebIDL, nor have a Getter. So that might be a follow-up to resolve.  https://www.w3.org/TR/webauthn/#publickeycredential
Comment on attachment 8890115 [details] Bug 1384307 - Set WebAuthn PublicKeyCredential's "id" and "type" fields https://reviewboard.mozilla.org/r/161200/#review166546 Cool - lgtm. ::: dom/webauthn/tests/test_webauthn_loopback.html:54 (Diff revision 1) > - attestationObject: CBOR object > - clientDataJSON: serialized JSON > - clientExtensionResults: (not yet supported) > */ > > + is(aCredInfo.type, "public-key", "Credential type must be public-key") nit: to me it makes sense to check these in the same order as the comment, but no big deal
Attachment #8890115 - Flags: review?(dkeeler) → review+
Thanks for the review, David! Marking checkin-needed.
Assignee: nobody → jjones
Status: NEW → ASSIGNED
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/42013738df2a Set WebAuthn PublicKeyCredential's "id" and "type" fields r=keeler
You need to log in before you can comment on or make changes to this bug.