Re: [webauthn] Nothing required in PublicKeyCredentialEntity

https://w3c.github.io/webauthn/#createCredential step 3 says:

> If any of the name member of options.rp, the name member of options.user, the displayName member of options.user, or the id member of options.user are not present, return a TypeError simple exception.

Because `name` is required in both the `user` and `rp` fields, I think it is ok to mark it `required` in the base class. `id` wasn't marked `required` when it was in PublicKeyCredentialEntity because it was only required by one of the sub-dictionaries, but now it probably could be marked `required` in `PublicKeyCredentialUserEntity`.

-- 
GitHub Notification of comment by jyasskin
Please view or discuss this issue at https://github.com/w3c/webauthn/issues/587#issuecomment-334309730 using your GitHub account

Received on Wednesday, 4 October 2017 22:37:45 UTC