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

keys made with the RSAEncrypt usage flag should be usable for encrypt… #439

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

s-lanc44
Copy link

@s-lanc44 s-lanc44 commented Jun 2, 2023

keys made with the RSAEncrypt usage flag should be usable for encryption. keys nowadays are made with the RSAEncryptOrSign but older keys are not.

…ion. keys nowadays are made with the RSAEncryptOrSign but older keys are not.
@dkg
Copy link
Contributor

dkg commented Jun 14, 2023

While this code looks technically correct, it'd probably be better to include a test that shows that encrypting to such an old key works.

However, in the course of looking into making a test, i have to ask whether this is a concretely useful change, or just something that makes the OpenPGP ecosystem worse.

RSA encrypt-only and RSA sign-only keys were formally deprecated ("SHOULD NOT be generated") in RFC 4880, released in November 2007, over 15 years ago.

I don't know of any public key material that is that old that is a good idea to still encrypt to in practice.

For archival purposes, i can see why you might want a test that enables decryption of material with such a secret key, but in general i don't think it's a good idea to enable encryption now, in 2023.

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