Skip to content

add yubikey init offering#52

Merged
biox merged 3 commits intomainfrom
jes/yubikey
Mar 8, 2025
Merged

add yubikey init offering#52
biox merged 3 commits intomainfrom
jes/yubikey

Conversation

@biox
Copy link
Owner

@biox biox commented Feb 23, 2025

for #49

arcxio
arcxio previously requested changes Feb 23, 2025
@arcxio
Copy link
Collaborator

arcxio commented Feb 23, 2025

also it would be great to use yubikey in contrib/pa-rekey as well to make a switch to yubikey trivial

biox and others added 2 commits March 8, 2025 13:05
Co-authored-by: arĉi <arcxi@dismail.de>
Co-authored-by: arĉi <arcxi@dismail.de>
@biox
Copy link
Owner Author

biox commented Mar 8, 2025

@arcxio - I addressed your feedback in d9628e1 - i'm in favor of the prompt, just so we don't surprise users who happen to have a yubikey plugged in, as rare as that might be.

@arcxio arcxio dismissed their stale review March 8, 2025 19:24

LGTM pa-rekey can be adressed separately I suppose

@biox biox merged commit 27f5563 into main Mar 8, 2025
@biox
Copy link
Owner Author

biox commented Mar 8, 2025

sounds good to me - i'll see if i can get to it back when im on my desktop w/ the yubikey

@biox biox mentioned this pull request Mar 8, 2025
arcxio added a commit that referenced this pull request Aug 1, 2025
* add yubikey init offering

* set proper age-plugin-yubikey params by default

Co-authored-by: arĉi <arcxi@dismail.de>

* address arcxio's feedback

Co-authored-by: arĉi <arcxi@dismail.de>

---------

Co-authored-by: arĉi <arcxi@dismail.de>
@arcxio arcxio deleted the jes/yubikey branch August 18, 2025 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants