mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2026-02-05 00:29:40 +03:00
Cannot register U2F key #115
Reference in New Issue
Block a user
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 @odensc on GitHub (Oct 27, 2018).
When I go to register a U2F key in the web vault, it errors on
POST /api/two-factor/get-u2f-challenge.@mprasil commented on GitHub (Oct 27, 2018):
Can you post the whole error log message?
@jmmco commented on GitHub (Oct 28, 2018):
I'm getting the same error logs from the container:
This is after clicking the Read Key in the FIDO U2F modal.
UPDATE:
I tried
1.3.0-alpineand was able to register the key then go toalpineand use the key. Seems there's a new step in the U2F registration.@dani-garcia commented on GitHub (Oct 28, 2018):
It seems the web vault has changed the APIs used for u2f, probably in version 2.4.0 when support for multiple keys was implemented.
For now, if you need to use u2f keys, you can use bitwarden_rs 1.2.0, which uses an older web vault version that should hopefully work.
Hopefully next week I'll have some time to investigate further and fix this.
@pwlgrzs commented on GitHub (Nov 2, 2018):
I presume this is related, I wanted to activate Duo on my instance but getting this:
@dani-garcia commented on GitHub (Nov 2, 2018):
Duo support is not implemented at the moment
@RomanHargrave commented on GitHub (Nov 8, 2018):
The "/api/two-factor/got-u2f" endpoint appeared in jslib at 4b7962d - "add support for u2f setup apis"
@RomanHargrave commented on GitHub (Nov 9, 2018):
I have tested and confirmed that at minimum, adding the new endpoint is enough to make u2f enrollment and authentication function.
Of course, this all is still single-key.
@RomanHargrave commented on GitHub (Nov 9, 2018):
I have a PR in the works (branch
multiple-u2fon my fork). Presently, it only re-adds support for single-key u2f; however, I would like to support the multi-key functionality in the Vault.@dani-garcia commented on GitHub (Nov 9, 2018):
The PR has been merged now, so I'll close this. If anyone finds another problem, please open another issue.