-
Notifications
You must be signed in to change notification settings - Fork 1.3k
fix: Add zlib/libzstd to base-debian13 image #1933
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
fix: Add zlib/libzstd to base-debian13 image #1933
Conversation
because OpenSSL 3.2+ uses those libraries to support TLS Certificate Compression, OpenSSL now requires zlib and libzstd. without this change, you need to add zlib/libzstd manually in your Dockerfile for use OpenSSL, otherwise you will get errors like `libzstd.so.1: cannot open shared object file: No such file or directory`. see: * https://salsa.debian.org/debian/openssl/-/commit/ca7e7b08746d408da112fe63d5a3bf941c345ec0 (note: Debian is disabling brotil compression, see https://salsa.debian.org/debian/openssl/-/commit/26109eddd699cce5c3aecb42ec1c5178cbde7d84 )
Summary of ChangesHello @rinsuki, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request adds zlib1g and libzstd1 to the base-debian13 image, as they are now required dependencies for OpenSSL 3.2+ for TLS certificate compression. The zlib1g package is also correctly moved from the cc image packages to the base image packages to ensure it's available in the base and avoid duplication. The changes are consistent across the configuration files and the generated lock files. I've added a couple of minor suggestions to improve maintainability by sorting the new package lists alphabetically.
private/repos/deb/trixie.yaml
Outdated
| - zlib1g | ||
| - libzstd1 | ||
| - openssl-provider-legacy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
What I Tested: Previously ( After change base image to this patch ( |
loosebazooka
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this looks fine mostly.
base/config.bzl
Outdated
| "libssl3t64", | ||
| "libzstd1", | ||
| "zlib1g", | ||
| "openssl-provider-legacy", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer not to include legacy algorithms
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done @ b3bb521
e0faed8 to
fe88732
Compare
|
Thanks! I'll merge this when tests pass |
because OpenSSL 3.2+ uses those libraries to support TLS Certificate Compression, OpenSSL now requires zlib and libzstd (see https://packages.debian.org/trixie/libssl3t64 ).
without this change, user need to add zlib/libzstd manually in their Dockerfile for use OpenSSL, otherwise they would get errors like
libzstd.so.1: cannot open shared object file: No such file or directory.see:
(note: Debian disables brotil compression so we don't need brotil libraries at this time, see https://salsa.debian.org/debian/openssl/-/commit/26109eddd699cce5c3aecb42ec1c5178cbde7d84 )
This should fix #1887 and other comments, e.g.
p.s. I couldn't run the full test suite since I dont have a enough free space to run that.