Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow newly created Passkeys to be added to an existing entry #10372

Closed
t4moxjc7 opened this issue Mar 10, 2024 · 6 comments · Fixed by #10408
Closed

Allow newly created Passkeys to be added to an existing entry #10372

t4moxjc7 opened this issue Mar 10, 2024 · 6 comments · Fixed by #10408

Comments

@t4moxjc7
Copy link

Summary

It would be useful if passkeys could be added to existing entries, rather than having to create separate entries.

Current workaround: exporting passkey from the new entry, importing that into the existing entry, then deleting the new entry.

Examples

If you already have an account in the database (e.g. with password auth) and want to add passkey auth to it.

Context

Passkey support added v2.7.7.

@droidmonkey
Copy link
Member

droidmonkey commented Mar 10, 2024

Sorry I misread. You want to use an existing entry at the point of Passkey creation. Good point, that is currently not possible. Nor is it possible to merge two entries (different issue) or directly migrate passkey data from one entry to another.

@droidmonkey droidmonkey changed the title Feature Request: Allow passkeys to be added to existing entry Allow newly created Passkeys to be added to an existing entry Mar 10, 2024
@droidmonkey droidmonkey added this to the v2.8.0 milestone Mar 10, 2024
@varjolintu varjolintu self-assigned this Mar 10, 2024
@varjolintu
Copy link
Member

Maybe we could add a new button to the dialog that opens the Import Dialog instead (with a different title of course).

@droidmonkey
Copy link
Member

droidmonkey commented Mar 10, 2024

That is what I was thinking too, could rename that dialog to "Choose Entry for Passkey" or something similar. Technically it is post-import.

@juvannx
Copy link

juvannx commented Apr 26, 2024

I think this merge should be done by browser extension.
I don't think there is any other way to enter passkeys into keepassxc.
However, developing issue #2202 could also help a lot, once the passkey has been created you can merge with an existing item.
At the moment it's quite boring, after enabling the passkey on github I had to copy all the attributes of the existing item into new item

@varjolintu
Copy link
Member

@juvannx The merge cannot be done in the extension side. See the linked PR in this issue for a solution.

@juvannx
Copy link

juvannx commented Apr 26, 2024

@varjolintu my fault, I described my suggestion in a wrong way, your PR does exactly what I wanted, I'll wait for version 2.7.8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants