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

generate: Add checksum validation for SSL modules #163

Merged
merged 2 commits into from
Jul 3, 2024

Conversation

dgl
Copy link
Contributor

@dgl dgl commented Jul 2, 2024

cpanm defaults to downloading via http, so currently while there is a checksum for App::cpanminus there isn't a checksum or validation for the SSL modules installed.

Rather than just switching to HTTPS, explicitly download the modules and verify the checksums, so the Docker image is built with known versions and checksums.

This obviously has the downside that future releases will need to be tracked manually, but I think that's a desirable thing in some ways, as the build of this docker image then becomes more reproducible.

cpanm defaults to downloading via http, so currently while there is a
checksum for App::cpanminus there isn't a checksum or validation for the
SSL modules installed.

Rather than just switching to HTTPS, explicitly download the modules and
verify the checksums, so the Docker image is built with known versions
and checksums.
generate.pl Show resolved Hide resolved
Copy link
Member

@zakame zakame left a comment

Choose a reason for hiding this comment

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

Hi @dgl, thanks for the PR! 🙏

Updated it to add the generate.pl outputs, looks good! Will get this published alongside the newest perldevel release.

@zakame zakame merged commit f7f4330 into Perl:master Jul 3, 2024
46 checks passed
zakame added a commit to zakame/docker-library-official-images that referenced this pull request Jul 3, 2024
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.

None yet

2 participants