pretalx / pretalx-docker

Docker setup for a complete pretalx installation. Community-sourced, not officially supported.
29 stars 39 forks source link

BREAKING CHANGE: reduce container size by removing Nginx #43

Closed MikkCZ closed 1 year ago

MikkCZ commented 1 year ago

Binds the port 80 directly to Gunicorn. For local development, Nginx inside the container is an overkill and an extra indirection. For production deployment one should setup the container behind a reverse proxy to handle TLS anyway.

I marked this as breaking change, because someone might have been configuring the Nginx inside the container to also handle TLS by mounting the certs and custom configuration file inside the container. However that's not a good practice, as deployment is then not correctly scalable.