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

Before importing an OpenPGP certificate, lint it #2052

Merged
merged 1 commit into from
May 12, 2022

Conversation

nwalfield
Copy link
Contributor

When importing an OpenPGP certificate, lint the certificate to show
the user possible issues. Fail if the certificate is completely
unusable. Using the Sequoia backend, this yields, for instance:

$ ./rpmkeys --import tests/data/keys/alice-revoked-subkey.asc
Certificate B3A771BFEB04E625:
Subkey 1F71177215217EE0 was revoked: Key material has been compromised, it was the maid
Certificate does not have any usable signing keys

Fixes #1974.

@nwalfield
Copy link
Contributor Author

Note: this change includes a stub implementation for the internal OpenPGP parser, whose feature development is frozen; this feature only works with the Sequoia backend.

@nwalfield nwalfield force-pushed the fix-1974 branch 2 times, most recently from 7cd6e6a to d47d43a Compare May 4, 2022 13:25
lib/rpmts.c Outdated Show resolved Hide resolved
lib/rpmts.c Outdated Show resolved Hide resolved
When importing an OpenPGP certificate, lint the certificate to show
the user possible issues.  Fail if the certificate is completely
unusable.  Using the Sequoia backend, this yields, for instance:

  $ ./rpmkeys --import tests/data/keys/alice-revoked-subkey.asc
  Certificate B3A771BFEB04E625:
    Subkey 1F71177215217EE0 was revoked: Key material has been compromised, it was the maid
    Certificate does not have any usable signing keys

Fixes rpm-software-management#1974.
@pmatilai
Copy link
Member

Okay, thanks!

@pmatilai pmatilai merged commit d703160 into rpm-software-management:master May 12, 2022
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.

Importing a key has no verbosity information, unhelpful error message
2 participants