github/dokku
0
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

fix: correct issue with detection of non-https port mapping as https port mapping

Browse Source 
This allows folks to _only_ set the letsencrypt-email in order to get https working as expected.

Closes #5410
This commit is contained in:
Jose Diaz-Gonzalez
2022-11-28 01:11:56 -05:00
parent 2f79e68542
commit b7bff8e152
4 changed files with 160 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
plugins/caddy-vhosts/docker-args-process-deploy
View File

@@ -72,6 +72,16 @@ trigger-caddy-vhosts-docker-args-process-deploy() {
fi
done
letsencrypt_email="$(fn-caddy-letsencrypt-email)"
if [[ -n "$letsencrypt_email" ]] && [[ -z "$proxy_container_https_port" ]]; then
proxy_container_https_port_candidate="$proxy_container_http_port_candidate"
proxy_host_https_port_candidate="$proxy_host_http_port_candidate"
if [[ -n "$proxy_container_http_port" ]]; then
proxy_container_https_port_candidate="$proxy_container_http_port"
proxy_host_http_port_candidate=443
fi
fi
app_domains="$(plugn trigger domains-list "$APP")"
if [[ -n "$app_domains" ]]; then
caddy_domains="$(echo "$app_domains" | xargs)"
@@ -98,7 +108,6 @@ trigger-caddy-vhosts-docker-args-process-deploy() {
fi
fi
letsencrypt_email="$(fn-caddy-letsencrypt-email)"
scheme="http"
if [[ -n "$letsencrypt_email" ]] && [[ "$has_443_mapping" == "true" ]]; then
output="--label 'caddy=${caddy_domains}'"