Check for user verifying platform authenticator (UVPA) #75
Conversation
|
This is branched off #74 so it has that work in it too, hence the WIP in the title. Also I want more time to look at this, I just did it today and I'm getting sleepy. |
peaceful-james
force-pushed
the
check-for-user-verifying-platform-authenticator
branch
from
360641a
to
35d5db8
Compare
This can be bypassed by simply deleting the
```
check_user_verifying_platform_authenticator_available={true}
```
line, or setting it to `false`
Co-authored-by: Owen Bickford <[email protected]>
peaceful-james
force-pushed
the
check-for-user-verifying-platform-authenticator
branch
from
9635529
to
dcded60
Compare
|
@type1fool I am considering renaming |
peaceful-james
changed the title
peaceful-james
changed the title
That seems like a good use of an acronym. |
priv/static/registration_hook.js
Outdated
| this.handleEvent("registration-challenge", (event) => | ||
| this.handleRegistration(event, this) | ||
| ); | ||
| }, | ||
| async checkUserVerifyingPlatformAuthenticatorAvailable(context) { | ||
| if (!(await window.PublicKeyCredential?.isUserVerifyingPlatformAuthenticatorAvailable())) { | ||
| const error = new Error("Registration unavailable. Your device does not support passkeys. Please install a passkey authenticator.") |
There was a problem hiding this comment.
I'd like to give devs the ability to tailor this message to their user base. To do that, we may need to move the message into the LiveView. What do you think?
There was a problem hiding this comment.
OK I did this with another data- html attribute.
peaceful-james
deleted the
check-for-user-verifying-platform-authenticator
branch
This introduces a hitherto unused check for "user-verifying platform authenticator".