Can't create a passkey in a password manager on Android if resident key is not required
Categories
(Core :: DOM: Web Authentication, defect, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox139 | --- | fixed |
People
(Reporter: mozilla.qns16, Assigned: michel)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
I have tried to test the support for Passkeys on the latest Firefox Build. More specifically I'm interested in the support for Third Party Password Managers.
Scenario:
- Make sure a third party password manager is set up as an autofill provider and passkey provider in the system (Phone Settings).
- Open the Firefox browser.
- Navigate to https://webauthn.io
- Try to register a passkey.
Expectations:
- The CredentialManager calls the default Credential provider (third party password manager) and allows to generate a passkey.
What happened:
- The Google Password Manager appears.
However, if I try to authenticate with a passkey, then the proper credential provider is invoked.
I have been able to replicate this behaviour with many password managers (Proton Pass, 1Password...)
Tested in Firefox Stable 128 and Firefox Beta 128.0b9, both in an Android 14 emulator, a Pixel 7a with Android 14, and on a Samsung Galaxy A14 with Android 14.
| Comment hidden (metoo) |
| Comment hidden (metoo) |
Comment 3•1 year ago
|
||
Current implementation is that we use Credential manager when "Discoverable Credential" is required. Default of webauthn.io is preferred.
| Comment hidden (metoo) |
Comment 5•1 year ago
|
||
The severity field is not set for this bug.
:jschanck, could you have a look please?
For more information, please visit BugBot documentation.
Comment 6•1 year ago
|
||
(In reply to Makoto Kato [:m_kato] from comment #3)
Current implementation is that we use Credential manager when "Discoverable Credential" is required. Default of webauthn.io is preferred.
When it's set to preferred, I get the "old" dialog which only supports Google Password Manager and not third-party applications. I assume this is a bug, and that preferred should use the Credential Manager the same way as if it is set to required?
| Comment hidden (metoo) |
Comment 8•1 year ago
|
||
I try to test about this.
-
Maybe this checker need to change from
!authenticatorSelection.getString("residentKey", "").equals("required")toauthenticatorSelection.getString("residentKey", "").equals("discouraged")
https://hg.mozilla.org/integration/autoland/file/7095d3bca694f62a308c408848f706b7a1e9a8d2/mobile/android/geckoview/src/main/java/org/mozilla/gecko/WebAuthnCredentialManager.java#l139 -
In Chromium, it will have two default provider.
https://chromium.googlesource.com/chromium/src/+/refs/heads/main/components/webauthn/android/java/src/org/chromium/components/webauthn/cred_man/GpmCredManRequestDecorator.java#58
Updated•1 year ago
|
| Assignee | ||
Comment 9•1 year ago
|
||
@m_kato why are you only using Credential Manager when residentKey is set to required instead of always attempting to use it and falling back to GMS if it is not available?
Comment 10•1 year ago
|
||
When implementing CM support, most 3rd party providers aren't available yet. So I didn't know whether these providers supported "preferred".
(First version of some credential managers crashes with some unknown parameters. So I thought it was better to disable without required.) But I guess that we can allow CM even if "preferred" now.
| Assignee | ||
Comment 11•1 year ago
|
||
Do you remember which versions were affected by this issue?
Comment 12•1 year ago
|
||
Although I don't know why you ask version (Android? Firefox?), I doesn't have strong reason not to allow "preferred" for credential manager provider support.
Comment 13•1 year ago
|
||
(And when we had implemented credential manager support in GeckoView, Chrome's support was completed yet. But Chrome supports it now. So I think that 3rd party provider has better credential manager support.)
| Assignee | ||
Updated•1 year ago
|
| Assignee | ||
Updated•1 year ago
|
| Assignee | ||
Updated•1 year ago
|
| Assignee | ||
Comment 14•1 year ago
|
||
Updated•11 months ago
|
Comment 15•11 months ago
|
||
Comment 16•11 months ago
|
||
| bugherder | ||
Updated•10 months ago
|
Description
•