fix(deploy): docker

Author: RustyRory

app/api/server.js

@@ -301,8 +301,8 @@ app.get('/api/diagnostics/networks', requireAuth, async (req, res) => {
 app.post('/api/deploy/apps/:name/full-restart', requireAuth, async (req, res) => {
   const { name } = req.params;
   try {
-    res.json({ ok: true, restarting: true });
-    composeFullRestart(name).catch((err) => console.error(`[full-restart] ${name} failed:`, err.message));
+    const result = await composeFullRestart(name);
+    res.json({ ok: true, ...result });
   } catch (err) {
     res.status(500).json({ error: err.message });
   }

app/api/services/compose.js

@@ -116,27 +116,37 @@ export async function composeRebuild(serviceNames, forceBuild = false) {
 
 export async function composeFullRestart(appName) {
   const allServices = await getAllServiceNames(appName);
-  console.log(`[full-restart] ${appName}: services détectés = [${allServices.join(', ')}]`);
+  console.log(`[full-restart] ${appName}: services = [${allServices.join(', ')}]`);
 
-  // Suppression de tous les containers de l'app (y compris bases de données)
+  // Suppression de tous les containers
   await execFile('docker', ['compose', '-f', MAIN_COMPOSE, 'rm', '-sf', ...allServices], { cwd: APPS_ROOT }).catch(() => {});
   await Promise.all(allServices.map((n) => execFile('docker', ['rm', '-f', n]).catch(() => {})));
 
-  // Suppression des réseaux orphelins liés à l'app
+  // Suppression puis recréation d'un réseau dédié propre
   const networkName = `${appName}-net`;
-  await execFile('docker', ['network', 'rm', networkName, `${appName}_net`], {}).catch(() => {});
-
-  // Création d'un réseau dédié pour l'app
-  await execFile('docker', ['network', 'create', networkName], {}).catch(() => {});
+  await execFile('docker', ['network', 'rm', networkName], {}).catch(() => {});
+  await execFile('docker', ['network', 'create', networkName], {});
 
   // Démarrage de tous les containers
   await execFile('docker', ['compose', '-f', MAIN_COMPOSE, 'up', '-d', ...allServices], { cwd: APPS_ROOT });
 
-  // Connexion explicite de tous les containers au réseau partagé (quel que soit leur réseau actuel)
-  await Promise.all(
-    allServices.map((n) => execFile('docker', ['network', 'connect', networkName, n]).catch(() => {}))
+  // Attente que les containers soient créés (3s)
+  await new Promise((r) => setTimeout(r, 3000));
+
+  // Connexion forcée de TOUS les containers au réseau partagé
+  const connectResults = await Promise.all(
+    allServices.map(async (n) => {
+      try {
+        await execFile('docker', ['network', 'connect', networkName, n]);
+        return `${n}: connecté`;
+      } catch (e) {
+        return `${n}: ${e.message.includes('already') ? 'déjà connecté' : e.message.split('\n')[0]}`;
+      }
+    })
   );
-  console.log(`[full-restart] ${appName}: tous les containers connectés à ${networkName}`);
+
+  console.log(`[full-restart] réseau ${networkName}:`, connectResults);
+  return { services: allServices, network: networkName, connections: connectResults };
 }
 
 const INFRA_COMPOSE_CONTENT = `services:

app/public/app.js

@@ -296,8 +296,16 @@ async function fullRestartApp(name, btn) {
   btn.disabled = true;
   btn.textContent = '…';
   const res = await fetch(`/api/deploy/apps/${encodeURIComponent(name)}/full-restart`, { method: 'POST' });
-  btn.textContent = res.ok ? '✅' : '❌';
-  setTimeout(() => { btn.disabled = false; btn.textContent = 'Redémarrage complet'; loadDeploy(); }, 3000);
+  const data = await res.json().catch(() => ({}));
+  if (res.ok) {
+    const info = data.connections ? `\n${data.connections.join('\n')}` : '';
+    alert(`✅ Redémarrage complet de "${name}" terminé.\nRéseau : ${data.network || '?'}${info}`);
+  } else {
+    alert(`❌ Erreur : ${data.error || 'inconnue'}`);
+  }
+  btn.disabled = false;
+  btn.textContent = 'Redémarrage complet';
+  loadDeploy();
 }
 
 function toggleEnvSection() {