Bug 213934
| Summary: | [WebAuthn] pinToken and uv option are mutually exclusive in CTAP2.0 | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | login Llama <loginllama> |
| Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | jiewen_tan, loginllama, nuno.sung, pascoe, tim.cappalli, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | iPhone / iPad | ||
| OS: | Other | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=181943 | ||
login Llama
iOS 14
Using a usb attached authenticator that has a getInfo options member that includes both clientPin=True and uv=True, causes the platform to first getPINToken then do makeCredential and getAssertion with both pinToken and the uv=1 option. This is valid in CTAP 2.1 for a three factor authentication.
In that flow the platform first prompts for clientPin then performs getPINToken.
It then performs getAssertion with the pinToken and uv=True to force the authenticator to do uv (typically fingerprint) the authenticator must validate both before returning a assertion.
So while technically allowed it is probably not the intent to have iOS do a three factor authentication.
I would expect for a authenticator that has a getInfo options member that includes both clientPin=True and uv=True in CTAP2.0. (The flow changes in CTAP2.1 but lets start with this.)
1) perform getAssertion or makeCredential with the uv option set to true.
2) If addassertion is returned then return to RP
3) If operation-denied is returned then error
4) If pin-required is returned fall back to client PIN
5) If the getInfo options member doesn't include clientPin=True Then error
6) prompt user for PIN
7) Negotiate shared secret
8) Do getPintoken
9) redo getAssertion with pinToken
10) joy
I skipped some steps depending on the size of the allow list. There is info on maximum credentialID size and max number of credentials in getInfo now that was added in FIDO_2_1_Pre that you should be using as well.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/65085265>
Jiewen Tan
Let's revisit this once we start implementing CTAP 2.1.
login Llama
CTAP 2.1 RD was updated so that if the platform sends getAssertion with both pinToken and the uv=1 option, then the uv=1 option is ignored.
We worked around the problem on the spec/authenticator side.
You should however still fix the problem
pascoe@apple.com
This was fixed in https://bugs.webkit.org/show_bug.cgi?id=232501 and https://bugs.webkit.org/show_bug.cgi?id=213903
login Llama
Thanks, I am seeing the correct behavior now in Safari 15.2 on OSX and iOS 15.2