-
Notifications
You must be signed in to change notification settings - Fork 26
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
Enable OpenSSL legacy providers support #194
base: main
Are you sure you want to change the base?
Conversation
Added the enable-legacy option for OpenSSL across PHP versions. This ensures that legacy providers will be compiled and could be enabled when needed.
Is it worth defining the modules path in the layer so there’s one less step? I think most places referencing enabling the legacy providers document the conf environment but not the module path so it could catch some people out |
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'm not sure this is a good idea to include for all Bref users. It may be better for you to compile your own image(s).
OpenSSL builds the legacy provider by default so in a way this provider is usually distributed in most operating systems that provide it and is kind of already being built by Bref but is being actively excluded so kind of shipping a non-default setup. (See - https://github.com/openssl/openssl/blob/openssl-3.3.1/INSTALL.md#no-legacy note above says if no-xxx is documented than the default is enable - so the enable-legacy in this PR is unnecessary at the moment.) I think if the default was to not build legacy then I would agree as then Bref is not actively delivering something weak. But I think as the default it to ship it it probably makes sense. Perhaps it just needs the “enable-legacy” removing so it automatically stops shipping when OpenSSL switches the default as at that point everywhere will have to rebuild their own. Just at moment it’s like everywhere is fine except Bref as it actively does something nothing else does |
@@ -120,6 +120,7 @@ RUN CFLAGS="" \ | |||
--openssldir=${INSTALL_DIR}/bref/ssl \ | |||
--release \ | |||
enable-tls1_3 \ | |||
enable-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 think this don’t exist in this one as it OpenSSL 1.1.1w and not 3
Overview
This pull request introduces a modification to the build process to include the legacy module (
legacy.so
) in the OpenSSL installation. This change is essential for enabling certain legacy ciphering algorithms that might still be required for specific integrations.Key Changes
enable-legacy
flag in the OpenSSL build process. This flag allows OpenSSL to compile and include legacy cryptographic algorithms.legacy.so
file is now copied into the final build. This enables runtime configuration of OpenSSL to use legacy ciphers.Important Notes
OPENSSL_MODULES
to point OpenSSL to the location of additional modules:OPENSSL_MODULES="/opt/lib/ossl-modules"
OPENSSL_CONF
environment variable:OPENSSL_CONF="{path to custom openssl config file}"
Impact
By including the legacy module and providing instructions on how to enable legacy algorithms, this change ensures that applications requiring older encryption methods can function correctly without significant changes to their cryptographic dependencies.