Bug 191516 – [WebAuthn] Support CTAP Client Pin

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Jiewen Tan]

We should consider support CTAP client pin command:https://fidoalliance.org/specs/fido-v2.0-ps-20170927/fido-client-to-authenticator-protocol-v2.0-ps-20170927.html#authenticatorClientPIN

[Radar WebKit Bug Importer]

<rdar://problem/56558558>

[login Llama]

Yes for keys with PIN set you cant create new credentials without the user inputting the PIN in CTAP2.   The workaround that Firefox and Chrome use when UV is set to discouraged is to make the credential using U2F on keys that support both CTAP1 and CTAP2.  For a CTAP2 only key or if you are making a resident credential there is no way to do it without PIN support.

I will note that currently CTAP2.1 will allow the creation of non resident credentials via CTAP2, but there are millions of authenticators in the market currently that will only support CTAP2.0.

[Jiewen Tan]

*** Bug 205222 has been marked as a duplicate of this bug. ***

[login Llama]

My first choice is supporting client pin. 
Second choice is using the U2F make credential workaround that Google used in Chrome.

On the other hand bug 204111 is the highest priority as it is breaking all CTAP 2 authenticators without builtin UV.  That is about 98%.  Until that gets fixed we cant really test this.

Thanks

[Jiewen Tan]

Created attachment 386914 [details]
Patch

[Jiewen Tan]

Created attachment 386984 [details]
Patch

[Jiewen Tan]

Created attachment 387005 [details]
Patch

[Jiewen Tan]

Created attachment 387028 [details]
Patch

[Brent Fulgham]

Comment on attachment 387028 [details]
Patch

r=me

[Jiewen Tan]

Thanks Brent for r+ this patch.

[Jiewen Tan]

Created attachment 387399 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 387399 [details]
Patch for landing

Rejecting attachment 387399 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 387399, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as [e-mail]...
Fetching: https://bugs.webkit.org/attachment.cgi?id=387399&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=191516&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Updating working directory
Processing patch 387399 from bug 191516.
Fetching: https://bugs.webkit.org/attachment.cgi?id=387399
Authentication required
Authentication required
Authentication required
Updating OpenSource
From https://git.webkit.org/git/WebKit
   8cd2c8b4c7e..e4cf8500860  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 254386 = 8cd2c8b4c7e20fe5b4f3240a3e719722a80784d9
r254387 = e4cf8500860fe817dd55f3a86a67f2a8ab032ead
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: https://webkit-queues.webkit.org/results/13302981

[Jiewen Tan]

Committed r254439: <https://trac.webkit.org/changeset/254439>

[Don Olmstead]

Reopening for build fix.

Commit queue cancelled my patch because of the following

In file included from /Volumes/Data/EWS/WebKit/WebKitBuild/Release/DerivedSources/WebKit2/unified-sources/UnifiedSource55.cpp:1:
In file included from /Volumes/Data/EWS/WebKit/Source/WebKit/UIProcess/WebAuthentication/fido/FidoService.cpp:31:
/Volumes/Data/EWS/WebKit/Source/WebKit/UIProcess/WebAuthentication/fido/CtapAuthenticator.h:66:70: error: unknown type name 'CryptoKeyEC'; did you mean 'WebCore::CryptoKeyEC'?
    void continueGetPinTokenAfterRequestPin(const String& pin, const CryptoKeyEC&);
                                                                     ^~~~~~~~~~~
                                                                     WebCore::CryptoKeyEC
/Volumes/Data/EWS/WebKit/Source/WebKit/UIProcess/WebAuthentication/fido/CtapAuthenticator.h:40:7: note: 'WebCore::CryptoKeyEC' declared here
class CryptoKeyEC;
      ^
1 error generated.

From https://webkit-queues.webkit.org/results/13303609 which came from https://bugs.webkit.org/show_bug.cgi?id=205964

[Don Olmstead]

Created attachment 387598 [details]
Build fix

[Don Olmstead]

Comment on attachment 387598 [details]
Build fix

Affected builds seem happy

[WebKit Commit Bot]

Comment on attachment 387598 [details]
Build fix

Clearing flags on attachment: 387598

Committed r254495: <https://trac.webkit.org/changeset/254495>

[Jiewen Tan]

(In reply to Don Olmstead from comment #16)
> Comment on attachment 387598 [details]
> Build fix
> 
> Affected builds seem happy

Thanks for fixing the issue.

[Don Olmstead]

(In reply to Jiewen Tan from comment #18)
> (In reply to Don Olmstead from comment #16)
> > Comment on attachment 387598 [details]
> > Build fix
> > 
> > Affected builds seem happy
> 
> Thanks for fixing the issue.

👍

[Jiewen Tan]

*** Bug 207821 has been marked as a duplicate of this bug. ***