From 715035e5dca11e6e4fd7c14b3c8a4dd59db2c01c Mon Sep 17 00:00:00 2001 From: Roland Date: Thu, 9 Jul 2026 15:57:57 -0600 Subject: [PATCH] help.sh: port 80 is the Controller's, not Caddy's; fix IPv4 port regex. The Caddyfile only binds 443/8084/8085/8086 and reverse-proxies to the Controller on port 80, which the Controller health check already covers. The IPv4 alternative in the port grep was missing its colon, so only the IPv6 ":::" listeners ever matched. Co-Authored-By: Claude Fable 5 --- help.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/help.sh b/help.sh index e85cb012..7c1c46d5 100755 --- a/help.sh +++ b/help.sh @@ -371,8 +371,9 @@ else echo "FAILED: Certificate renewal timer is not enabled (fix with: systemctl enable --now wrolpi-cert-renew.timer)" fi -for port in 80 443 8084 8085 8086; do - if netstat -ant | grep LISTEN | grep -E "(0\.0\.0\.0|:::)${port}\b" >/dev/null; then +# Port 80 belongs to the Controller (checked above); Caddy proxies to it. +for port in 443 8084 8085 8086; do + if netstat -ant | grep LISTEN | grep -E "(0\.0\.0\.0:|:::)${port}\b" >/dev/null; then echo "OK: Caddy port ${port} is occupied" else echo "FAILED: Caddy port ${port} is not occupied"