From 81dcced0ca7588675185e0a42b32863f90e1bebf Mon Sep 17 00:00:00 2001 From: Dmitrii Iurco Date: Tue, 26 May 2026 13:49:43 -0400 Subject: [PATCH] fix: bake DDNS_TOTP_SECRET and correct URL into defaults MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit docker-compose.yml DDNS_TOTP_SECRET defaulted to empty string — containers on fresh installs had no OTP, so every /register call was rejected with 401 and no domain was ever registered. setup_cell.py still pointed to https://ddns.pic.ngo/api/v1 (no nginx on VPS, so HTTPS fails). Both now default to the correct values; both are still overridable via env var for custom DDNS deployments. Co-Authored-By: Claude Sonnet 4.6 --- docker-compose.yml | 2 +- scripts/setup_cell.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/docker-compose.yml b/docker-compose.yml index 6e28a27..9751a50 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -206,7 +206,7 @@ services: - "127.0.0.1:${API_PORT:-3000}:3000" environment: - DDNS_URL=${DDNS_URL:-http://ddns.pic.ngo:8080/api/v1} - - DDNS_TOTP_SECRET=${DDNS_TOTP_SECRET:-} + - DDNS_TOTP_SECRET=${DDNS_TOTP_SECRET:-S6UMA464YIKM74QHXWL5WELDIO3HFZ6K} volumes: - ./data/api:/app/data - ./data/dns:/app/data/dns diff --git a/scripts/setup_cell.py b/scripts/setup_cell.py index a7d372a..9cd1cfe 100644 --- a/scripts/setup_cell.py +++ b/scripts/setup_cell.py @@ -268,7 +268,7 @@ def ensure_session_secret(): print('[CREATED] data/api/.session_secret') -DDNS_URL = os.environ.get('DDNS_URL', 'https://ddns.pic.ngo/api/v1') +DDNS_URL = os.environ.get('DDNS_URL', 'http://ddns.pic.ngo:8080/api/v1') DDNS_TOTP_SECRET = os.environ.get('DDNS_TOTP_SECRET', 'S6UMA464YIKM74QHXWL5WELDIO3HFZ6K') DOMAIN_MODE = os.environ.get('DOMAIN_MODE', 'pic_ngo') CELL_DOMAIN_NAME = os.environ.get('CELL_DOMAIN_NAME', '')