Skip to content

MCLOUD-6469: dh key too small when trying to use magento-cloud-docker… #277

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

Merged
merged 6 commits into from
Aug 19, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ jobs:
env:
- TEST_SUITE=build-images
script:
- if [ $TRAVIS_SECURE_ENV_VARS == "true" ]; then ./tests/travis/images.sh tls; fi;
- if [ $TRAVIS_SECURE_ENV_VARS == "true" ]; then ./tests/travis/images.sh varnish; fi;
- if [ $TRAVIS_SECURE_ENV_VARS == "true" ]; then ./tests/travis/images.sh elasticsearch; fi;
- if [ $TRAVIS_SECURE_ENV_VARS == "true" ]; then ./tests/travis/images.sh nginx; fi;
Expand Down
14 changes: 0 additions & 14 deletions images/nginx/1.9/nginx-healthcheck.sh

This file was deleted.

3 changes: 0 additions & 3 deletions images/php/7.2-fpm/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -51,9 +51,6 @@ RUN wget https://github.com/mailhog/mhsendmail/releases/download/v0.2.0/mhsendma
&& sudo chmod +x mhsendmail_linux_amd64 \
&& sudo mv mhsendmail_linux_amd64 /usr/local/bin/mhsendmail

# Change security level to avoid `dh key too small` error
RUN sed -i 's/SECLEVEL=2/SECLEVEL=1/g' /etc/ssl/openssl.cnf

# Configure the gd library
RUN docker-php-ext-configure \
gd --with-freetype-dir=/usr/include/ --with-jpeg-dir=/usr/include/
Expand Down
3 changes: 0 additions & 3 deletions images/php/7.3-fpm/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,6 @@ RUN wget https://github.com/mailhog/mhsendmail/releases/download/v0.2.0/mhsendma
&& sudo chmod +x mhsendmail_linux_amd64 \
&& sudo mv mhsendmail_linux_amd64 /usr/local/bin/mhsendmail

# Change security level to avoid `dh key too small` error
RUN sed -i 's/SECLEVEL=2/SECLEVEL=1/g' /etc/ssl/openssl.cnf

# Configure the gd library
RUN docker-php-ext-configure \
gd --with-freetype-dir=/usr/include/ --with-jpeg-dir=/usr/include/
Expand Down
3 changes: 0 additions & 3 deletions images/php/7.4-fpm/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,6 @@ RUN wget https://github.com/mailhog/mhsendmail/releases/download/v0.2.0/mhsendma
&& sudo chmod +x mhsendmail_linux_amd64 \
&& sudo mv mhsendmail_linux_amd64 /usr/local/bin/mhsendmail

# Change security level to avoid `dh key too small` error
RUN sed -i 's/SECLEVEL=2/SECLEVEL=1/g' /etc/ssl/openssl.cnf

# Configure the gd library
RUN docker-php-ext-configure \
gd --with-freetype=/usr/include/ --with-jpeg=/usr/include/
Expand Down
3 changes: 0 additions & 3 deletions images/php/fpm/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,6 @@ RUN wget https://github.com/mailhog/mhsendmail/releases/download/v0.2.0/mhsendma
&& sudo chmod +x mhsendmail_linux_amd64 \
&& sudo mv mhsendmail_linux_amd64 /usr/local/bin/mhsendmail

# Change security level to avoid `dh key too small` error
RUN sed -i 's/SECLEVEL=2/SECLEVEL=1/g' /etc/ssl/openssl.cnf

# Configure the gd library
{%docker-php-ext-configure%}

Expand Down
25 changes: 0 additions & 25 deletions images/tls/Dockerfile

This file was deleted.

50 changes: 0 additions & 50 deletions images/tls/certs/magento2.docker.pem

This file was deleted.

29 changes: 0 additions & 29 deletions images/tls/entrypoint.sh

This file was deleted.

73 changes: 0 additions & 73 deletions images/tls/pound.cfg

This file was deleted.

16 changes: 6 additions & 10 deletions images/nginx/1.9/Dockerfile → images/web/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,9 +1,11 @@
FROM nginx:1.9
FROM nginx

ENV UPLOAD_MAX_FILESIZE 64M
ENV FPM_HOST fpm
ENV XDEBUG_HOST fpm_xdebug
ENV FPM_PORT 9000
ENV VARNISH_HOST web
ENV VARNISH_PORT 80
ENV MAGENTO_ROOT /app
ENV MAGENTO_RUN_MODE production
ENV MFTF_UTILS 0
Expand All @@ -13,26 +15,20 @@ COPY etc/nginx.conf /etc/nginx/
COPY etc/vhost.conf /etc/nginx/conf.d/default.conf
COPY etc/xdebug-upstream.conf /etc/nginx/conf.d/xdebug/upstream.conf

RUN apt-get update && apt-get install -y openssl curl
RUN mkdir /etc/nginx/ssl \
&& echo -e "\n\n\n\n\n\n\n" | openssl req -x509 -nodes -days 365 -newkey rsa:2048 -keyout /etc/nginx/ssl/magento.key -out /etc/nginx/ssl/magento.crt
RUN mkdir /etc/nginx/ssl
COPY certs/* /etc/nginx/ssl/

RUN groupadd -g 1000 www && useradd -g 1000 -u 1000 -d ${MAGENTO_ROOT} -s /bin/bash www

VOLUME ${MAGENTO_ROOT}

COPY nginx-healthcheck.sh /usr/local/bin/nginx-healthcheck.sh
RUN ["chmod", "+x", "/usr/local/bin/nginx-healthcheck.sh"]

HEALTHCHECK --retries=3 CMD ["bash", "/usr/local/bin/nginx-healthcheck.sh"]

COPY docker-entrypoint.sh /docker-entrypoint.sh
RUN ["chmod", "+x", "/docker-entrypoint.sh"]
ENTRYPOINT ["/docker-entrypoint.sh"]

USER root

EXPOSE 80
EXPOSE 443

WORKDIR ${MAGENTO_ROOT}

Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ XDEBUG_UPSTREAM_FILE="/etc/nginx/conf.d/xdebug/upstream.conf"
[ ! -z "${XDEBUG_HOST}" ] && sed -i "s/!XDEBUG_HOST!/${XDEBUG_HOST}/" $XDEBUG_UPSTREAM_FILE
[ ! -z "${FPM_PORT}" ] && sed -i "s/!FPM_PORT!/${FPM_PORT}/" $VHOST_FILE
[ ! -z "${FPM_PORT}" ] && sed -i "s/!FPM_PORT!/${FPM_PORT}/" $XDEBUG_UPSTREAM_FILE
[ ! -z "${VARNISH_HOST}" ] && sed -i "s/!VARNISH_HOST!/${VARNISH_HOST}/" $VHOST_FILE
[ ! -z "${VARNISH_PORT}" ] && sed -i "s/!VARNISH_PORT!/${VARNISH_PORT}/" $VHOST_FILE
[ ! -z "${MAGENTO_ROOT}" ] && sed -i "s#!MAGENTO_ROOT!#${MAGENTO_ROOT}#" $VHOST_FILE
[ ! -z "${MAGENTO_RUN_MODE}" ] && sed -i "s/!MAGENTO_RUN_MODE!/${MAGENTO_RUN_MODE}/" $VHOST_FILE
[ ! -z "${MFTF_UTILS}" ] && sed -i "s/!MFTF_UTILS!/${MFTF_UTILS}/" $VHOST_FILE
Expand Down
File renamed without changes.
21 changes: 20 additions & 1 deletion images/nginx/1.9/etc/vhost.conf → images/web/etc/vhost.conf
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,27 @@ upstream fastcgi_backend {
}

server {
listen 80;
listen 443 ssl;

server_name _;

ssl_certificate /etc/nginx/ssl/magento.crt;
ssl_certificate_key /etc/nginx/ssl/magento.key;

location / {
proxy_pass http://!VARNISH_HOST!:!VARNISH_PORT!;
proxy_set_header Host $host;
proxy_set_header X-Real-IP $remote_addr;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
proxy_set_header X-Forwarded-Proto $scheme;
proxy_buffer_size 128k;
proxy_buffers 4 256k;
proxy_busy_buffers_size 256k;
}
}

server {
listen 80;

fastcgi_buffers 16 16k;
fastcgi_buffer_size 32k;
Expand Down
38 changes: 24 additions & 14 deletions src/Compose/ProductionBuilder.php
Original file line number Diff line number Diff line change
Expand Up @@ -255,23 +255,27 @@ public function build(Config $config): Manager
[self::NETWORK_MAGENTO],
[self::SERVICE_DB => ['condition' => 'service_healthy']]
);

$webConfig = [
'volumes' => $volumesRo,
'environment' => [
'WITH_XDEBUG=' . (int)$config->hasServiceEnabled(ServiceInterface::SERVICE_FPM_XDEBUG)
]
];

if (!$config->hasServiceEnabled(self::SERVICE_VARNISH)) {
$webConfig['ports'] = [$config->getPort() . ':80'];
$webConfig['environment'][] = 'VIRTUAL_HOST=' . $config->getHost();
$webConfig['environment'][] = 'VIRTUAL_PORT=' . $config->getPort();
$webConfig['environment'][] = 'HTTPS_METHOD=noredirect';
}

$manager->addService(
self::SERVICE_WEB,
$this->serviceFactory->create(
ServiceInterface::SERVICE_NGINX,
$config->getServiceVersion(ServiceInterface::SERVICE_NGINX),
[
'volumes' => $volumesRo,
'environment' => [
'VIRTUAL_HOST=' . $config->getHost(),
'VIRTUAL_PORT=80',
'HTTPS_METHOD=noredirect',
'WITH_XDEBUG=' . (int)$config->hasServiceEnabled(ServiceInterface::SERVICE_FPM_XDEBUG)
],
'ports' => [
$config->getPort() . ':80'
]
]
$webConfig
),
[self::NETWORK_MAGENTO],
[self::SERVICE_FPM => []]
Expand All @@ -288,11 +292,17 @@ public function build(Config $config): Manager
self::NETWORK_MAGENTO => [
'aliases' => [$config->getHost()]
]
],
'ports' => [$config->getPort() . ':80'],
'environment' => [
'VIRTUAL_HOST=' . $config->getHost(),
'VIRTUAL_PORT=' . $config->getPort(),
'HTTPS_METHOD=noredirect'
]
]
),
[],
[self::SERVICE_WEB => ['condition' => 'service_healthy']]
[self::SERVICE_WEB => ['condition' => 'service_started']]
);
}

Expand All @@ -305,7 +315,7 @@ public function build(Config $config): Manager
ServiceInterface::SERVICE_TLS,
$config->getServiceVersion(ServiceInterface::SERVICE_TLS),
[
'environment' => ['HTTPS_UPSTREAM_SERVER_ADDRESS' => $tlsBackendService],
'environment' => ['VARNISH_HOST' => $tlsBackendService],
]
),
[self::NETWORK_MAGENTO],
Expand Down
Loading