-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
Feature: PGP Asymmetric Encryption #456
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this valuable addition. I left a few comments inline, nothing major. Let me know what you think.
use correct env var in example Co-authored-by: Frederik Ring <frederik.ring@gmail.com>
use errwarp for initial error msg Co-authored-by: Frederik Ring <frederik.ring@gmail.com>
Thank you for reviewing. I just hope all this is getting squashed 🙂 |
This looks good to me now. There's a tiny complaint by the linter still, but once that's fixed I'd say this is good to go. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the feature and the conversation, that's much appreciated. I'll (squash) merge this when #457 is ready so I can cut a release for these changes then. I'll let you know when that happens 🎩
This is now included in v2.42.0 |
While this project already provides symmetric pgp encryption, it lacks a way to encrypt backups asymmetric.
Therefore this pull request provides that functionality.
Therefore the
encryptArchive
function has been changed. All changes have been tested and documented.I have not done much programming in Golang in the last year. Suggestions for improvement are welcome.