From ea5b796012f6ed8b333b07b119c5dfbedd8da69d Mon Sep 17 00:00:00 2001
From: mm <mm@kalina>
Date: Mon, 14 Sep 2026 10:42:21 +0200
Subject: [PATCH] =?UTF-8?q?fix(installer):=20envsubst-Redirection=20wieder?=
 =?UTF-8?q?hergestellt=20=E2=80=94=20DAS=20war=20der=20x14-Haenger?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Der Trace auf x14 endete exakt hier:
  + envsubst ${NODE_NAME} ... ${APACHE_ERROR_FILES}
  root@x14:/home/mm#

Ohne "< \$2.tmp" liest envsubst STDIN. Bei sh install.sh vom Terminal ist stdin
die Konsole -> der Installer wartet auf Eingabe, nach der letzten Meldung
"Schreibe Konfiguration ...". Genau das Bild: Stillstand, ^C, kein Fehler.
Bei curl | sh waere es schlimmer: envsubst wuerde den Rest des Skripts fressen.

Ein frueherer Patch von mir hat die Redirection verschluckt; alle Laeufe mit
vorhandener telegraf.conf (x22) liefen trotzdem, weil dort nur der Fallback
subst_vars() aktiv war. Erst der Erstinstall (x14) traf envsubst.

Verifiziert: vorher Haenger bei envsubst, jetzt vollstaendiger Durchlauf mit
CONFIG_VERSION-Marker und allen Confs.
---
 install.sh | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/install.sh b/install.sh
index c677f60..7b4a64b 100755
--- a/install.sh
+++ b/install.sh
@@ -815,7 +815,15 @@ fetch_conf() {
   fetch "$1" "$2.tmp"
   load_defaults
   if command -v envsubst >/dev/null; then
-    envsubst '${NODE_NAME} ${PROM_URL} ${LOKI_URL} ${TELEGRAF_TOKEN} ${PVE_TOKEN} ${PVE_NODE} ${PVE_URL} ${APACHE_LOG} ${APACHE_FILES} ${APACHE_ERROR_FILES}'
+    # PFLICHT: Redirection. Ohne "< $2.tmp" liest envsubst STDIN.
+    #
+    # Realer Ausfall (x14): hier fehlte die Redirection. `envsubst '${…}'` ohne
+    # Eingabeumleitung wartet auf stdin — bei `sh install.sh` vom Terminal ist
+    # das die Konsole, der Installer blieb also NACH "Schreibe Konfiguration …"
+    # stehen und wartete auf Eingabe. Der Trace endete exakt hier.
+    # Bei `curl … | sh` waere es noch schlimmer: envsubst wuerde den REST des
+    # Skripts als Eingabe verbrauchen.
+    envsubst '${NODE_NAME} ${PROM_URL} ${LOKI_URL} ${TELEGRAF_TOKEN} ${PVE_TOKEN} ${PVE_NODE} ${PVE_URL} ${APACHE_LOG} ${APACHE_FILES} ${APACHE_ERROR_FILES}' < "$2.tmp" > "$2"
   else
     subst_vars "$2.tmp" "$2" NODE_NAME PROM_URL LOKI_URL TELEGRAF_TOKEN PVE_TOKEN PVE_NODE PVE_URL APACHE_LOG APACHE_FILES APACHE_ERROR_FILES
   fi