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

add pa-rekey #27

Merged
merged 1 commit into from
Jun 19, 2024
Merged

add pa-rekey #27

merged 1 commit into from
Jun 19, 2024

Conversation

arcxio
Copy link
Collaborator

@arcxio arcxio commented Jun 19, 2024

No description provided.

@arcxio arcxio mentioned this pull request Jun 19, 2024
Copy link
Contributor

@mbauhardt mbauhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From my perspective a re-key is not only replacing a key.
adding keys is a use case as well.

Copy link
Contributor

@mbauhardt mbauhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's a bit weird to me to have only the option to replace all keys, and not adding a key, but if the end user know and understand the script he/she can manually copy the identities and recipients file with the extension .tmp to get this feature.

naja. the user-story or let's say the use-case I had in mind isn't really solved but I can help myself.

@arcxio
Copy link
Collaborator Author

arcxio commented Jun 19, 2024

the user-story or let's say the use-case I had in mind isn't really solved but I can help myself

wdyt about this interface? you use it like this:

PA_IDENTITIES=~/.local/share/pa/identities PA_RECIPIENTS=~/.local/share/pa/recipients contrib/pa-rekey

and it will do the copying for you

@mbauhardt
Copy link
Contributor

Perfect. This will satisfy my needs! thx. lets merge it.

@arcxio arcxio merged commit 81da389 into main Jun 19, 2024
@arcxio arcxio deleted the rekey branch June 19, 2024 12:17
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