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

Verify PGP key embedded in .sources files are correct #1183

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

legoktm
Copy link
Member

@legoktm legoktm commented Sep 24, 2024

Status

Ready for review

Description of Changes

It's kind of hard to manually test the PGP key inside apt's .sources files are the ones we intended, so write a test to do it. This is copied from the one I wrote in securedrop-client, and adds coverage for the apt-test sources file.

This isn't a launcher test, but it needs to have external dependencies installed, so this is the best place for it now until we have unit tests for the rest of the provisioning code.

Testing

  • CI passes
  • Visual review (key fingerprint in test is correct)

Deployment

Any special considerations for deployment? n/a

It's kind of hard to manually test the PGP key inside apt's .sources
files are the ones we intended, so write a test to do it. This is copied
from the one I wrote in securedrop-client, and adds coverage for the
apt-test sources file.

This isn't a launcher test, but it needs to have external dependencies
installed, so this is the best place for it now until we have unit tests
for the rest of the provisioning code.
Copy link
Contributor

@zenmonkeykstop zenmonkeykstop left a comment

Choose a reason for hiding this comment

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

LGTM

@zenmonkeykstop zenmonkeykstop added this pull request to the merge queue Sep 30, 2024
Merged via the queue into main with commit 1172a9d Sep 30, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants