[webauthn] Closed Pull Request: remove "required" from PublicKeyCredentialDescriptor.id

equalsJeffH has just closed AngeloKai's pull request 500 for https://github.com/w3c/webauthn:

== remove "required" from PublicKeyCredentialDescriptor.id  ==
Addresses https://github.com/w3c/webauthn/issues/245

1. Remove the "required" keyword from PublicKeyCredentialDescriptor.id, which is a member of the allowCredentials and excludeCredentials dictionary.
2. Add requirements to the algorithm that "if the credential type is 'PublicKeyCredential', the id is required."

The benefit of the PR is that we keep a door open for future credential types. If there are multiple credential types, the developer can select credentials based on types.


<!--
    This comment and the below content is programatically generated.
    You may add a comma-separated list of anchors you'd like a
    direct link to below (e.g. #idl-serializers, #idl-sequence):

    Don't remove this comment or modify anything below this line.
    If you don't want a preview generated for this pull request,
    just replace the whole of this comment's content by "no preview"
    and remove what's below.
-->
***
[Preview](https://s3.amazonaws.com/pr-preview/AngeloKai/webauthn/angelo-removeRequired.html) | [Diff](https://s3.amazonaws.com/pr-preview/w3c/webauthn/119dd51...AngeloKai:495a464.html)

See https://github.com/w3c/webauthn/pull/500

Received on Friday, 19 April 2019 15:31:57 UTC