mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2026-04-25 17:25:57 +03:00
[GH-ISSUE #1796] Cannot add second security key #1062
Labels
No labels
SSO
Third party
better for forum
bug
bug
documentation
duplicate
enhancement
future Vault
future Vault
future Vault
good first issue
help wanted
low priority
notes
pull-request
question
troubleshooting
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/vaultwarden#1062
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @fireglow on GitHub (Jun 26, 2021).
Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/1796
Subject of the issue
When trying to add a second security key in the Two-step Login FIDO2 WebAuthn UI, an error occurs.
vaultwarden version:
1.21.0-46e0f3c4
Install method:
cargo build --releaseClients used:
Steps to reproduce
Expected behaviour
The second key is added to the list
Actual behaviour
The UI returns an error ("There was a problem reading the security key. Try again.") immediately after clicking on the "Read Key" button.
The browser console logs
TypeError: CredentialsContainer.create: 'transports' member of PublicKeyCredentialDescriptor can't be converted to a sequence.The vaultwarden logs only show the expected
Troubleshooting data
@BlackDex commented on GitHub (Jun 26, 2021):
Confirmed.
There already is a PR to fix this upstream: https://github.com/kanidm/webauthn-rs/pull/80
@BlackDex commented on GitHub (Jun 26, 2021):
The PR has been merged. So we just need to update the commit hash, test it, and merge that on our side.
@fireglow commented on GitHub (Jun 27, 2021):
Can confirm that with the webauthn-rs commit updated to 02a99f534127b30c6f4df7f2d42bc24f76dc4211 in Cargo.toml,
it's possible to add additional security keys.