Re: [webauthn] Initial proposal for modifications to resident key credential requirements (#1191)

@sbweeden 
> I have addressed both the pick-ups, and found one additional dfn reference error which I fixed as well.

Yep, looks good.  changing 'credential' to 'pk cred source' is good catch, thx :)
Would be good to add the comma and fix grammar error that @emlun caught.

@emlun 
> With the changes @agl proposes in fido-alliance/fido-2-specs#661 it looks to me like we actually could support forbidden here. Should we? (We could do that in a separate PR if we don't want to hold this one up for it)

you mean add "forbidden" as a fourth value of [ResidentKeyRequirement](file:///usr/local/google/home/jdhodges/_stds/w3c/webauthn/equalsjeffh/fork-sbweeden-webauthn/index.html#enumdef-residentkeyrequirement) ?

-- 
GitHub Notification of comment by equalsJeffH
Please view or discuss this issue at https://github.com/w3c/webauthn/pull/1191#issuecomment-489139387 using your GitHub account

Received on Friday, 3 May 2019 15:39:06 UTC