Adding :- Syntax which ensures that, if SKIP_CADDY is unset, it will be treated as an empty string, which will not match "true". This avoids potential issues with unbound variables in some shell configurations where set -u (treating unset variables as an error) is enabled.
(ChatGPT)
Starting the docker container is hampered due to setting `chown -R www-data:www-data /var/www/html/` on every start, causing it to traverse the entire directory which in our use case is very slow. This PR instead changes it to set permissions as part of the build process.
Sidenote: Is `LE_EMAIL` supposed to be used in addition to `ADMIN_EMAIL`?
* enable installer on docker first run
* add SESSION_COOKIE to compose file
* `APP_ENVIRONMENT_ONLY` is long gone
* session env vars no longer needed after #624
* set defaults to null if sqlite is selected
Update the dockerfile to use supervisord
Update supervisord config to use start caddy unless configured not to.
Updated entrypoint to handle caddy skip for supervisord.
dockerfile
cmd updated to just start php-fpm
entrypoint
now starts caddy unless SKIP_CADDY has been set.
compose file
updated ports to work properly.
updated networks to use the correct network.
added commented port and variable to disable caddy
added further notes.
Currently container startup will ignore any errors, which will tend
to leave things in a broken state if operations like migrations or
certificate provisioning fail. Prefer to terminate the container
rather than try to limp on.
