From 1b17b39b5de8d864b7165d2863bf61e1c53e7eba Mon Sep 17 00:00:00 2001
From: Manus <manus@santinova-soft.org>
Date: Mon, 17 Aug 2026 20:20:44 +0000
Subject: [PATCH] refactor: fiabiliser le portail dynamique recette

---
 index.html                   | 189 ++++++++++++++++++++---------------
 test/portal-contract.test.js |  21 ++++
 2 files changed, 129 insertions(+), 81 deletions(-)
 create mode 100644 test/portal-contract.test.js

diff --git a/index.html b/index.html
index a2da238..614923c 100644
--- a/index.html
+++ b/index.html
@@ -375,111 +375,138 @@
   </footer>
 
   <script>
-    // URL de l'API du dashboard de recette
+    /*
+     * Configuration d'environnement. Cette version est exclusivement destinée
+     * à la recette. La version production ne diffère que par ces quatre valeurs.
+     */
+    var ENV = 'recette';
+    var ENV_LABEL = 'Recette';
     var DASHBOARD_APPS_URL = 'https://dashboard.recette.santinova-soft.org/api/public/apps';
     var DASHBOARD_STATUS_URL = 'https://dashboard.recette.santinova-soft.org/api/public/status';
+    var STATUS_REFRESH_MS = 30000;
+    var APPS_REFRESH_MS = 300000;
+    var hasRenderedApps = false;
 
-    function switchTab(tabName, btn) {
-      document.querySelectorAll('.tab-btn').forEach(function(b) { b.classList.remove('active'); });
-      document.querySelectorAll('.tab-panel').forEach(function(p) { p.classList.remove('active'); });
-      btn.classList.add('active');
+    function switchTab(tabName, button) {
+      document.querySelectorAll('.tab-btn').forEach(function(tab) { tab.classList.remove('active'); });
+      document.querySelectorAll('.tab-panel').forEach(function(panel) { panel.classList.remove('active'); });
+      button.classList.add('active');
       document.getElementById('panel-' + tabName).classList.add('active');
     }
 
-    /**
-     * Construit une tuile HTML pour une application.
-     */
+    function escapeHtml(value) {
+      return String(value || '').replace(/[&<>'"]/g, function(character) {
+        return ({ '&': '&amp;', '<': '&lt;', '>': '&gt;', "'": '&#39;', '"': '&quot;' })[character];
+      });
+    }
+
+    function isHttpsUrl(value) {
+      try {
+        return new URL(value).protocol === 'https:';
+      } catch (error) {
+        return false;
+      }
+    }
+
+    function isOffline(app) {
+      return app.status === 'offline' || app.status === 'error' || app.containerRunning === false;
+    }
+
+    /** Construit une tuile depuis un manifeste validé par le dashboard local. */
     function buildCard(app) {
-      var url = (app.urls && app.urls.recette) ? app.urls.recette : '#';
+      var url = app.urls && app.urls[ENV];
+      if (!isHttpsUrl(url)) return '';
+
       var image = app.image || ('images/' + app.id + '.png');
-      var isOffline = app.status === 'offline' || !app.containerRunning;
-      var offlineClass = isOffline ? ' offline' : '';
-      return '<a class="app-card' + offlineClass + '" data-app-id="' + app.id + '" href="' + url + '" target="_blank" rel="noopener noreferrer">' +
-        '<div class="card-image-wrapper"><img src="' + image + '" alt="' + app.name + '" onerror="this.src=\'images/default.png\'" /></div>' +
-        '<div class="card-name">' + app.name + '</div>' +
-        '<div class="card-env-badge">Recette</div>' +
+      var offline = isOffline(app);
+      var offlineClass = offline ? ' offline' : '';
+      var ariaDisabled = offline ? ' aria-disabled="true"' : '';
+
+      return '<a class="app-card' + offlineClass + '" data-app-id="' + escapeHtml(app.id) + '" href="' + escapeHtml(url) + '" target="_blank" rel="noopener noreferrer"' + ariaDisabled + '>' +
+        '<div class="card-image-wrapper"><img src="' + escapeHtml(image) + '" alt="' + escapeHtml(app.name) + '" onerror="this.style.display=\'none\'" /></div>' +
+        '<div class="card-name">' + escapeHtml(app.name) + '</div>' +
+        '<div class="card-env-badge">' + ENV_LABEL + '</div>' +
         '<div class="card-status-badge">Hors ligne</div>' +
         '</a>';
     }
 
-    /**
-     * Charge les apps depuis l'API et construit les grilles.
-     */
-    function loadApps() {
-      fetch(DASHBOARD_APPS_URL, { cache: 'no-store' })
-        .then(function(r) {
-          if (!r.ok) throw new Error('HTTP ' + r.status);
-          return r.json();
-        })
-        .then(function(apps) {
-          var itinovaApps = apps.filter(function(a) { return a.category === 'ITINOVA'; });
-          var santinovaApps = apps.filter(function(a) { return a.category === 'SANTINOVA'; });
+    function renderApps(apps) {
+      // Défense en profondeur : le dashboard exclut INFRA et le portail le fait aussi.
+      var visibleApps = apps.filter(function(app) {
+        return app.category !== 'INFRA' && isHttpsUrl(app.urls && app.urls[ENV]);
+      });
+      var itinovaApps = visibleApps.filter(function(app) { return app.category === 'ITINOVA'; });
+      var santinovaApps = visibleApps.filter(function(app) { return app.category === 'SANTINOVA'; });
 
-          var gridI = document.getElementById('grid-itinova');
-          var gridS = document.getElementById('grid-santinova');
+      var gridItinova = document.getElementById('grid-itinova');
+      var gridSantinova = document.getElementById('grid-santinova');
+      gridItinova.innerHTML = itinovaApps.length
+        ? itinovaApps.map(buildCard).join('')
+        : '<div class="loading-msg">Aucune application Itinova déployée.</div>';
+      gridSantinova.innerHTML = santinovaApps.length
+        ? santinovaApps.map(buildCard).join('')
+        : '<div class="loading-msg">Aucune application Santinova déployée.</div>';
 
-          if (itinovaApps.length === 0) {
-            gridI.innerHTML = '<div class="loading-msg">Aucune application Itinova déployée.</div>';
-          } else {
-            gridI.innerHTML = itinovaApps.map(buildCard).join('');
-          }
-
-          if (santinovaApps.length === 0) {
-            gridS.innerHTML = '<div class="loading-msg">Aucune application Santinova déployée.</div>';
-          } else {
-            gridS.innerHTML = santinovaApps.map(buildCard).join('');
-          }
-
-          document.getElementById('count-itinova').textContent = itinovaApps.length;
-          document.getElementById('count-santinova').textContent = santinovaApps.length;
-        })
-        .catch(function(err) {
-          console.warn('[Portail] Impossible de charger les apps:', err.message);
-          document.getElementById('grid-itinova').innerHTML = '<div class="loading-msg">Erreur de chargement. Veuillez rafraîchir la page.</div>';
-          document.getElementById('grid-santinova').innerHTML = '<div class="loading-msg">Erreur de chargement. Veuillez rafraîchir la page.</div>';
-        });
+      document.getElementById('count-itinova').textContent = itinovaApps.length;
+      document.getElementById('count-santinova').textContent = santinovaApps.length;
+      hasRenderedApps = true;
     }
 
-    /**
-     * Met à jour l'état visuel des tuiles en fonction des statuts reçus du dashboard.
-     */
-    function applyStatuses(statuses) {
-      document.querySelectorAll('.app-card[data-app-id]').forEach(function(card) {
-        var appId = card.getAttribute('data-app-id');
-        var appStatus = statuses.find(function(s) { return s.id === appId; });
-        if (appStatus) {
-          var isOffline = appStatus.status === 'offline' || !appStatus.containerRunning;
-          if (isOffline) {
-            card.classList.add('offline');
-          } else {
-            card.classList.remove('offline');
-          }
+    async function requestJson(url) {
+      var controller = new AbortController();
+      var timeoutId = setTimeout(function() { controller.abort(); }, 8000);
+      try {
+        var response = await fetch(url, { cache: 'no-store', signal: controller.signal });
+        if (!response.ok) throw new Error('HTTP ' + response.status);
+        return await response.json();
+      } finally {
+        clearTimeout(timeoutId);
+      }
+    }
+
+    async function loadApps() {
+      try {
+        var apps = await requestJson(DASHBOARD_APPS_URL);
+        if (!Array.isArray(apps)) throw new Error('réponse applications invalide');
+        renderApps(apps);
+        await fetchAndApplyStatuses();
+      } catch (error) {
+        console.warn('[Portail] Chargement des applications impossible :', error.message);
+        if (!hasRenderedApps) {
+          var message = '<div class="loading-msg">Dashboard indisponible — veuillez réessayer.</div>';
+          document.getElementById('grid-itinova').innerHTML = message;
+          document.getElementById('grid-santinova').innerHTML = message;
+          document.getElementById('count-itinova').textContent = '!';
+          document.getElementById('count-santinova').textContent = '!';
         }
+      }
+    }
+
+    function applyStatuses(statuses) {
+      if (!Array.isArray(statuses)) return;
+      var byId = new Map(statuses.map(function(status) { return [status.id, status]; }));
+      document.querySelectorAll('.app-card[data-app-id]').forEach(function(card) {
+        var status = byId.get(card.getAttribute('data-app-id'));
+        if (!status) return;
+        var offline = isOffline(status);
+        card.classList.toggle('offline', offline);
+        card.setAttribute('aria-disabled', String(offline));
       });
     }
 
-    /**
-     * Interroge le dashboard pour obtenir les statuts et met à jour les tuiles.
-     */
-    function fetchAndApplyStatuses() {
-      fetch(DASHBOARD_STATUS_URL, { cache: 'no-store' })
-        .then(function(r) {
-          if (!r.ok) throw new Error('HTTP ' + r.status);
-          return r.json();
-        })
-        .then(applyStatuses)
-        .catch(function(err) {
-          console.warn('[Portail] Impossible de récupérer les statuts:', err.message);
-        });
+    async function fetchAndApplyStatuses() {
+      try {
+        applyStatuses(await requestJson(DASHBOARD_STATUS_URL));
+      } catch (error) {
+        // Conserver le dernier état connu est préférable à un grisage erroné.
+        console.warn('[Portail] Mise à jour des statuts impossible :', error.message);
+      }
     }
 
     document.addEventListener('DOMContentLoaded', function() {
-      // Charger les apps au démarrage
       loadApps();
-      // Polling statuts toutes les 30 secondes
-      setInterval(fetchAndApplyStatuses, 30000);
-      // Premier check statuts après 3s (laisser le temps aux tuiles d'être rendues)
-      setTimeout(fetchAndApplyStatuses, 3000);
+      setInterval(fetchAndApplyStatuses, STATUS_REFRESH_MS);
+      setInterval(loadApps, APPS_REFRESH_MS);
     });
   </script>
 </body>
diff --git a/test/portal-contract.test.js b/test/portal-contract.test.js
new file mode 100644
index 0000000..e1a3671
--- /dev/null
+++ b/test/portal-contract.test.js
@@ -0,0 +1,21 @@
+/*
+ * Contrat de non-régression du portail dynamique.
+ * Il bloque les marqueurs de conflit, les doublons de configuration
+ * et le mélange des URLs recette/production.
+ */
+const assert = require('node:assert/strict');
+const fs = require('node:fs');
+const path = require('node:path');
+
+const html = fs.readFileSync(path.join(__dirname, '..', 'index.html'), 'utf8');
+
+assert.equal(/^(<<<<<<<|=======|>>>>>>>)/m.test(html), false, 'aucun marqueur de conflit Git ne doit être livré');
+assert.equal((html.match(/var DASHBOARD_APPS_URL/g) || []).length, 1, 'une seule URL API applications doit être déclarée');
+assert.equal((html.match(/var DASHBOARD_STATUS_URL/g) || []).length, 1, 'une seule URL API statuts doit être déclarée');
+assert.match(html, /var ENV = 'recette';/, 'la version recette doit déclarer explicitement son environnement');
+assert.match(html, /app\.urls\[ENV\]/, 'les liens doivent être sélectionnés depuis l’environnement déclaré');
+assert.match(html, /app\.category !== 'INFRA'/, 'les tuiles INFRA doivent être filtrées côté portail');
+assert.match(html, /var STATUS_REFRESH_MS = 30000;/, 'le délai de grisage doit être explicite');
+assert.match(html, /setInterval\(fetchAndApplyStatuses, STATUS_REFRESH_MS\)/, 'le grisage doit être réactualisé périodiquement');
+
+console.log('OK portal-contract');