Repository navigation
fix(docker): remove ssl_client from the published images (#59) - #60
Closed
Xavier Fournet (xfournet) wants to merge 1 commit into
Closed
Xavier Fournet (xfournet) wants to merge 1 commit into
Xavier Fournet (xfournet) wants to merge 1 commit into
Conversation
ssl_client is the TLS helper of BusyBox wget (CVE-2025-60876, no fixed version) and is only needed to download sources during the build. Alpine installs it automatically next to busybox and libssl3, so 'apk del' keeps it: a '!ssl_client' world constraint removes it once the last wget download is done and keeps it out of the images built from these ones. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Member
Author
|
Closing: removing ssl_client does not fix CVE-2025-60876, which sits in the BusyBox wget applet itself, and the '!ssl_client' world constraint would outlive the fix in every image built from these ones. Nothing in these images runs wget on an untrusted URL; the next apk upgrade will pick up Alpine's fix once it ships (alpine/aports#17872). |
Xavier Fournet (xfournet)
deleted the
issue/59-fix-docker-remove-ssl-client-from-the
branch
November 6, 2026 21:11
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed changes
ssl_clientfrom the published images, legacy ones included. It is the TLS helper of BusyBoxwget(CVE-2025-60876, no fixed version), needed only to download the OpenSSL and Python sources during the build.ssl_clientautomatically next tobusyboxandlibssl3(install_if), soapk del ssl_clientkeeps it. The world constraintapk add --no-cache '!ssl_client'purges it, after the lastwgetdownload, and keeps it out of laterapk addandapk upgrade, including in the images built from these ones.wgetcan no longer fetch HTTPS URLs in these images and the images built from them. None of the current consumers uses it: the OpenCTI platform and worker FIPS images and the 16 connector FIPS images only runapk,gitandpip.Related issues
How to test this PR
podman build --target python-fips .and--target python-nodejs-fips .succeed, FIPS assertions included;openssl list -providersreports the FIPS provider 3.1.2.apk info -e ssl_clientfinds nothing,/etc/apk/worldholds!ssl_client, andssl_clientstays absent afterapk add --no-cache jq.mitreconnector'sDockerfile_fips, built on the newpython-fipsimage, installs its requirements (gitandpipfrom GitHub) and importspyctiandconnectors_sdk;ssl_clientstays absent.legacy/Dockerfile_pythonandlegacy/Dockerfile_python_nodejsbuild;ssl_clientis absent and the FIPS provider is listed.Checklist
type(scope?): description (#issue)🤖 Generated with Claude Code