-
Notifications
You must be signed in to change notification settings - Fork 890
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
Set GPG encrypt. conform. to RFC4880 Fixes #896 #924
base: main
Are you sure you want to change the base?
Conversation
Any update on when this fix would be released? This is currently blocking us from using sops. |
I'm not a SOPS maintainer, so cannot comment on when this will be merged. Since I'm a first-time contributed, or seem to "need a maintainer to approve running workflows" |
@ajvb @felixfontein Are you able to assist with kicking off the CI/CD workflow? "First-time contributors need a maintainer to approve running workflows." Cheers, |
@cameronkerrnz I'm not a maintainer either, I cannot trigger a run as wel... but @autrilla should be able to (next to @ajvb) :) |
@autrilla Could you please help. |
I approved the run. Before merging, I'd like a better explanation of why this is needed and what the impact is. Further, I think we should look for some other maintainers for sops, since I clearly do not have the time the project needs. I'll start a discussion for this. |
Analysis for why this is needed is in #896. Please let me know if there is anything insufficient there. The impact is most likely to be felt by people who have their keychains transported to different environments (eg. workstation upgrade, use of Docker in development (particular Visual Studio Code's Remote Containers, which forwards GPG material by default), multi-platform users. The previous (v2.2) behaviour was to use RFC4880, but v2.3 (at least in GnuPG distributed in Debian Buster, and container images based on that) is to default to RFC4880bis, which results in an incompatibility (something encrypted with RFC4880bis cannot be decrypted with RFC4880) I'm not personally using this version of GPG now (I decided to put this all into a container instead so my developers use a consistent version of GPG), but others have signalled that it is a blocking concern for them. Thanks, |
/LGTM |
@autrilla is there anything blocking this? we ran across this issue more often as 2.3 is installed more and more per default.... |
Resolves an issue where a file encrypted with GnuPG 2.3 which defaults to enabling RFC4880bis) can't be decrypted in earlier versions (eg. GnuPG 2.2 as used in Debian Buster)
I have tested encryption on GnuPG 2.3.1 on Windows, and subsequent decryption on GnuPG 2.2.12 on Debian Buster.
(this is recreation of #910 rebased onto the develop branch)