-
Notifications
You must be signed in to change notification settings - Fork 197
MCLOUD-6469: dh key too small when trying to use magento-cloud-docker-tls #273
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
Conversation
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.
Would the solution be to change how the SSL key and certificate are generated in tls.magento2.docker compared to changing the SECLEVEL in the PHP images? Doesn't changing SECLEVEL from 2 to 1 decrease security?
@mkozell I tried to generate RSA:2048, RSA:4096, and RSA:8192 but still have the same error.
Do you know how to generate certificate and keys which satisfies SECLEVEL=2 |
QA Approved |
It looks like a newer version of Pound may be needed for supporting larger DH keys. |
Description
Changed security level in
/etc/ssl/openssl.cnf
to avoiddh key too small
errorFixed Issues (if relevant)
https://jira.corp.magento.com/browse/MCLOUD-6469
Manual testing scenarios
docker exec -it magento-cloud_fpm_1 bash
curl -k https://tls.magento2.docker/
will return an errordh key too small
docker-compose down -v
docker build -t magento-cloud-docker-php-fpm-6469 images/php/7.4-fpm
docker-compose.yaml
filedocker-compose up -d
docker-compose run build cloud-build
docker-compose run deploy cloud-deploy
docker-compose run deploy cloud-post-deploy
docker exec -it magento-cloud_fpm_1 bash
curl -k https://tls.magento2.docker/
command run without errorRelease notes
Changed the security level in the OpenSSL configuration file (
/etc/ssl/openssl.cnf
) to fix adh key too small
error that occurs on TLS connection requests in the Cloud Docker environment.Associated documentation updates
Not required.
Contribution checklist