Merge pull request #12 from Mouette03/Fix-cockpit

Fix cockpit disable functionality

Author: Mouette03

stage-tipi/01-config/00-run.sh

@@ -60,10 +60,14 @@ systemctl enable tipi-runtipi-retry.service
 systemctl enable avahi-daemon.service
 
 # ---- Cockpit — interface web de gestion système (port 9090) ----
-# Installé ici, mais désactivé par défaut.
-# L'utilisateur peut l'activer via le portail de configuration (case à cocher).
+# Installé ici, mais masqué par défaut (disable seul ne suffit pas : les presets
+# APT/systemd peuvent le ré-activer au premier démarrage).
+# L'utilisateur peut l'activer via le portail de configuration (case à cocher) ;
+# setup.py fait un unmask + enable --now à ce moment-là.
 apt-get install -y cockpit
-systemctl disable cockpit.socket 2>/dev/null || true
+systemctl disable cockpit.socket  2>/dev/null || true
+systemctl mask    cockpit.socket  2>/dev/null || true
+systemctl mask    cockpit.service 2>/dev/null || true
 
 # ---- Activer SSH (désactivé par défaut sur Trixie) ----
 systemctl enable ssh.service

stage-tipi/01-config/files/app/setup.py

@@ -425,11 +425,23 @@ def get_final_ip() -> str | None:
 def configure_cockpit(enabled: bool):
     if enabled:
         step(T["cockpit_step"])
-        subprocess.run(["systemctl", "enable", "--now", "cockpit.socket"], check=False)
-        done(T["cockpit_done"])
-    else:
-        subprocess.run(["systemctl", "disable", "--now", "cockpit.socket"],
+        # Le socket et le service sont masqués au build — il faut d'abord
+        # lever le masque avant de pouvoir les activer.
+        subprocess.run(["systemctl", "unmask", "cockpit.socket", "cockpit.service"],
                        check=False, capture_output=True)
+        r = subprocess.run(["systemctl", "enable", "--now", "cockpit.socket"],
+                           capture_output=True, text=True)
+        if r.returncode != 0:
+            err(f"Cockpit : activation échouée — {(r.stderr or r.stdout).strip()}")
+        else:
+            done(T["cockpit_done"])
+    else:
+        r1 = subprocess.run(["systemctl", "disable", "--now", "cockpit.socket"],
+                            capture_output=True, text=True)
+        r2 = subprocess.run(["systemctl", "mask", "cockpit.socket", "cockpit.service"],
+                            capture_output=True, text=True)
+        if r1.returncode != 0 or r2.returncode != 0:
+            err("Cockpit : désactivation incomplète — vérifier manuellement")
 
 
 # ---------------------------------------------------------------------------