- From: Emil Lundberg via GitHub <sysbot+gh@w3.org>
- Date: Wed, 02 May 2018 11:18:24 +0000
- To: public-webauthn@w3.org
The following commits were just pushed by emlun to https://github.com/w3c/webauthn:
* Fix #848: Weirdness in RP UP verification
§7.1. Registering a new credential currently reads (and §7.2. Verifying
an authentication assertion is analogous):
>10. If user verification is required for this registration, verify that
the User Verified bit of the `flags` in _authData_ is set.
>11. If user verification is not required for this registration, verify
that the User Present bit of the `flags` in _authData_ is set.
This results in the following truth table:
UV req. | UP req? | UV res. | UP res. | Accept?
------------+---------+---------+---------+--------
discouraged | Yes | 0 | 0 | No
discouraged | Yes | 0 | 1 | Yes
discouraged | Yes | 1 | 0 | No
discouraged | Yes | 1 | 1 | Yes
preferred | Yes | 0 | 0 | No
preferred | Yes | 0 | 1 | Yes
preferred | Yes | 1 | 0 | No
preferred | Yes | 1 | 1 | Yes
required | No | 0 | 0 | No
required | No | 0 | 1 | No
required | No | 1 | 0 | Yes
required | No | 1 | 1 | Yes
Note, for example, how UV `preferred` means that a response with `(UV=1,
UP=0)` should be rejected.
It makes more sense to let the UP requirement be defined by the UV
_response_ instead of the UV _requirement_:
UV req. | UV res. | UP req? | UP res. | Accept?
------------+---------+---------+---------+--------
discouraged | 0 | Yes | 0 | No
discouraged | 0 | Yes | 1 | Yes
discouraged | 1 | No | 0 | Yes
discouraged | 1 | No | 1 | Yes
preferred | 0 | Yes | 0 | No
preferred | 0 | Yes | 1 | Yes
preferred | 1 | No | 0 | Yes
preferred | 1 | No | 1 | Yes
required | 0 | Yes | 0 | No
required | 0 | Yes | 1 | No
required | 1 | No | 0 | Yes
required | 1 | No | 1 | Yes
by Emil Lundberg
https://github.com/w3c/webauthn/commit/5b026ad5fcab52b853995a8ca8f4959bd9f9c0b7
* Always verify UP bit in RP ops
As pointed out by @akshayku, recent drafts of CTAP now support this.
by Emil Lundberg
https://github.com/w3c/webauthn/commit/a46247182ccbda508a619f27d4437649a30004a3
* Merge branch 'master' into issue-848-rp-up-verification
by JeffH
https://github.com/w3c/webauthn/commit/2ae16c671edfbce5dc6ec020deae706a13d751c9
* Fix |authData| -> |aData|
by Emil Lundberg
https://github.com/w3c/webauthn/commit/2dabcf8ac7a5734504443148697c69c86697a799
* Merge pull request #849 from w3c/issue-848-rp-up-verification
Fix #848: Weirdness in RP UP verification
by Emil Lundberg
https://github.com/w3c/webauthn/commit/c22b1ac49d3cbb4ba1902272ae5470bc2292ea55
Received on Wednesday, 2 May 2018 11:18:36 UTC