Re: [webauthn] credential ID returned by authenticatorGetAssertion() is optional if allowList has exactly one member

Thanks for the review & guidance @domenic !

@domenic noted in irc #whatwg that when he says "post a task", he means "[[=queue a task=]](https://html.spec.whatwg.org/#queue-a-task)" 

@domenic wrote:
> - The algorithm stays synchronous
> - Anywhere it says "in parallel" gets removed. It's doing all its work on the background thread already.

Ok.

> It never returns actual JS objects or messes with actual JS globals. Instead it returns the necessary data to create one. (Perhaps as an Infra struct.)

Ok, will look to https://github.com/WebAssembly/design/pull/1093 ([preview](https://github.com/WebAssembly/design/pull/1093/files?short_path=8e85308#diff-8e85308ab5cc1e83e91ef59233648be2)) for guidance. 

> - The "request a credential" operation, 

ah, you are referring to [webappsec-credential-management/#abstract-opdef-request-a-credential](https://w3c.github.io/webappsec-credential-management/#abstract-opdef-request-a-credential) here it seems.

> after synchronously retrieving the error code or data necessary to create a credential, [queues] a task (going "back to the main thread") to actually do the object/exception creation and resolve/reject the promise.

ok, though IIUC, [webappsec-credential-management/#abstract-opdef-request-a-credential](https://w3c.github.io/webappsec-credential-management/#abstract-opdef-request-a-credential) does not itself presently "queue a task...to actually do the object/exception creation and resolve/reject the promise", yes?

I am thinking that all the "object/exception creation and resolve/reject the promise" functionality could be specified in the webauthn spec in a fashion similar to [WebAssembly PR #1093](https://github.com/WebAssembly/design/pull/1093/files?short_path=8e85308#diff-8e85308ab5cc1e83e91ef59233648be2)'s approach. 

WDYT?

/cc @mikewest 



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

Received on Tuesday, 20 June 2017 22:42:17 UTC