Re: [webauthn] address empty allowlist in 'use existing cred' alg, fixes #387

thanks @rlin1 

> It fixes #481 in my opinion.

great, that issue ought to get auto-closed upon merge. 

> I think there is one issue left:
line 834 and line 839: credentialDescriptorList appears as parameter in the call to the Authenticator.
But: This appear inside our "in parallel for each credential C in credentialDescriptorList".

Yes, tho that is the topic of issue #480 which I am intending to work on addressing after this PR is merged.

I'll "move" your 2nd https://github.com/w3c/webauthn/pull/427#issuecomment-306543869 over to issue #480 

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

Received on Tuesday, 6 June 2017 18:04:10 UTC