xirixiz/dsmr-reader-docker

ENABLE_NGINX_SSL does not work anymore

arjanvanderveen opened this issue · 4 comments

arjanvanderveen commented

Support guidelines

I've found an issue and checked that ...

  • ... the documentation does not mention anything about my problem
  • ... there are no open or closed issues that are related to my problem

Description

When using the environment variable ENABLE_NGINX_SSL set to true, port 443 is not enabled in the Nginx config, due to a faulty sed command.

Expected behaviour

Nginx is listening on port 443

Actual behaviour

Nginx is listening on port 80

Steps to reproduce

Set the environment variable ENABLE_NGINX_SSL=true
Start the docker
The container is listening on port 80 and not port 443

Docker info

n/a

Version

  • Docker compose version (type docker-compose --version):
  • System info (type uname -a):
    Linux xirixiz-dsmr-reader 4.4.180+ #42962 SMP Sat Apr 8 00:14:23 CST 2023 x86_64 GNU/Linux

Docker compose

n/a

Container logs

n/a

Additional info

d7e7731#commitcomment-110488906

When i change the file /etc/cont-init.d/20-set-app-defaults back to the previous code, it works correctly.

xirixiz commented

Ja, ik denk dat ik het hele nginx config stuk eruit ga slopen. Het is denk ik beter dat wanneer je een specifieke config wenst, dat je die dan mount in de container. Het geknip en geplak met bash is onbetrouwbaar helaas.

Rolling back.

arjanvanderveen commented

Kan het me goed voorstellen. sed is tricky op momenten.

xirixiz commented

Afschuwelijk is het hahaha 😄

arjanvanderveen commented

Je kan ook nginx includes gebruiken ipv sed. Het is maar een ideetje hoor.