Oeffentliche Statusseite nach Statuspage-Vorbild

Neue Seite /status auf dem Hub: Gesamtlage als Banner, darunter die
freigegebenen Dienste mit 90-Tage-Balken, die Game-Server, und die
Stoerungs-Historie der letzten 90 Tage.

Die Adressliste ist von einer kommagetrennten Einstellung in eine Tabelle
gewandert. Jeder Dienst hat jetzt Namen, Gruppe und einen Schalter
"oeffentlich" — sonst stuende auf einer oeffentlichen Seite die nackte URL,
auch von internen Diensten. Die alte Liste wird beim ersten Start uebernommen,
bewusst als nicht oeffentlich: was dort auftaucht, soll eine Entscheidung sein
und nicht durch eine Migration passieren.

/api/status liefert nur freigegebene Dienste und dabei keine Adressen, nur
Namen. Auch die Stoerungs-Historie ist gefiltert — sonst verriete sie, was es
sonst noch gibt. Im Browser gegengeprueft: auf der ganzen Seite steht kein
einziges https://.

Stoerungen werden jetzt aufgezeichnet, nicht nur gemeldet: pro Adresse ein
offener Eintrag, bis es wieder laeuft. Beim Wiederkommen wird er geschlossen,
bevor die DM rausgeht — sonst bliebe er bei einem Sendefehler ewig offen.

Aufbewahrung von 7 auf 90 Tage. Der Balken kann jetzt zwei Aufloesungen:
168 Stunden fuers Panel ("was war letzte Nacht?") und 90 Tage fuer die
Statusseite ("wie zuverlaessig?"). Stuendlich ueber 90 Tage waeren ueber 2000
Striche und unlesbar.

Im Panel sind aus der reinen Anzeige Verwaltung geworden: anlegen, bearbeiten,
entfernen, freigeben — dazu die Stoerungsliste.

Beim Bauen gefunden: die Uebernahme der alten Liste stand mitten in db.js und
rief getSetting(), dessen prepared statement erst 300 Zeilen weiter unten
entsteht. Der Bot starb beim Start mit "Cannot access getSettingStmt before
initialization" — nachgewiesen, dann ans Dateiende verschoben.

Geprueft: 182 SQL-Abfragen gegen das Schema, alle schreibenden Routen bewacht,
Stoerungs-Logik durchgespielt (doppeltes Eroeffnen wird abgelehnt, Schliessen
ohne offene gibt null), und beide Seiten im Browser mit drei Diensten in allen
drei Zustaenden.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

src/bot/watchdog.js

@@ -1,7 +1,10 @@
 // Watchdog: prüft konfigurierte URLs, meldet Ausfälle per DM und schreibt
 // jeden Check mit — daraus baut die Übersicht im Panel ihren Uptime-Balken.
 // URLs kommen aus dem Setting watchdog_urls (Config-Seite), leer = deaktiviert.
-import { getSetting, recordWatchdogSample, pruneWatchdogHistory } from '../db.js';
+import {
+    listMonitored, recordWatchdogSample, pruneWatchdogHistory,
+    startIncident, endIncident, pruneIncidents,
+} from '../db.js';
 import { config } from '../config.js';
 import { moduleEnabled } from '../modules.js';
 import { brandEmbed } from '../embeds.js';
@@ -15,11 +18,10 @@ export function watchdogState() {
     return state;
 }
 
-export function watchedUrls() {
-    return (getSetting('watchdog_urls') ?? '')
-        .split(',')
-        .map((s) => s.trim())
-        .filter((s) => /^https?:\/\//.test(s));
+/** Was geprüft wird — kommt jetzt aus der Dienst-Tabelle, nicht mehr aus der
+ *  kommagetrennten Einstellung. Die wurde beim ersten Start übernommen. */
+export function watchedServices() {
+    return listMonitored().filter((s) => /^https?:\/\//.test(s.url));
 }
 
 async function dmAdmin(client, embed) {
@@ -31,7 +33,8 @@ async function dmAdmin(client, embed) {
 /** Ein Prüfdurchlauf — exportiert für Tests und den Intervall-Timer */
 export async function watchdogTick(client, { failsBeforeAlert = tuning('watchdog_fails') } = {}) {
     if (!moduleEnabled('watchdog')) return;
-    for (const url of watchedUrls()) {
+    for (const dienst of watchedServices()) {
+        const { url } = dienst;
         const s = state.get(url) ?? { fails: 0, down: false, since: null };
 
         let ok = false;
@@ -64,27 +67,31 @@ export async function watchdogTick(client, { failsBeforeAlert = tuning('watchdog
         if (ok) {
             if (s.down) {
                 const minuten = Math.max(1, Math.round((Date.now() - s.since) / 60000));
+                // Störung schließen, bevor gemeldet wird — sonst steht sie bei
+                // einem Fehler beim Senden für immer offen in der Historie.
+                endIncident(url);
                 await dmAdmin(client, brandEmbed(client, 'WATCHDOG')
                     .setTitle('✅ Wieder erreichbar')
-                    .setDescription(`**${url}**`)
+                    .setDescription(`**${dienst.name}**\n${url}`)
                     .addFields(
                         { name: 'Ausfall', value: `~${minuten} min`, inline: true },
                         { name: 'Antwort', value: `${detail} · ${dauer} ms`, inline: true },
                     ));
-                console.log(`[watchdog] ${url} wieder ok`);
+                console.log(`[watchdog] ${dienst.name} wieder ok`);
             }
             state.set(url, { fails: 0, down: false, since: null, status, ms: dauer, geprueft: Date.now() });
         } else {
             const fails = s.fails + 1;
             if (!s.down && fails >= failsBeforeAlert) {
+                startIncident(url, detail);
                 await dmAdmin(client, brandEmbed(client, 'WATCHDOG', { secondary: true })
                     .setTitle('🚨 Nicht erreichbar')
-                    .setDescription(`**${url}**`)
+                    .setDescription(`**${dienst.name}**\n${url}`)
                     .addFields(
                         { name: 'Fehler', value: detail, inline: true },
                         { name: 'Versuche', value: `${fails} in Folge`, inline: true },
                     ));
-                console.warn(`[watchdog] ${url} down (${detail})`);
+                console.warn(`[watchdog] ${dienst.name} down (${detail})`);
                 state.set(url, { fails, down: true, since: Date.now(), status, ms: null, geprueft: Date.now() });
             } else {
                 state.set(url, { ...s, fails, status, ms: null, geprueft: Date.now() });
@@ -92,9 +99,10 @@ export async function watchdogTick(client, { failsBeforeAlert = tuning('watchdog
         }
     }
 
-    // Alte Messpunkte wegräumen, gleiche Aufbewahrung wie beim Game-Server
+    // Alte Messpunkte und abgeschlossene Störungen wegräumen (90 Tage)
     try {
         pruneWatchdogHistory();
+        pruneIncidents();
     } catch { /* nicht kritisch, beim nächsten Durchlauf wieder */ }
 }
 

src/db.js

@@ -712,9 +712,23 @@ export const playerHistory = (serverId, hours = 24) => playerHistoryStmt.all(ser
 export const uptimeBuckets = (serverId, hours = 168) => uptimeBucketsStmt.all(serverId, `-${hours} hours`);
 export const prunePlayerHistory = () => prunePlayerHistoryStmt.run();
 
-// Watchdog: Erreichbarkeit der überwachten Adressen — gleiche Bauart wie oben,
-// damit die Übersicht denselben Balken benutzen kann. Adresse als Schlüssel:
-// die Liste steht als Einstellung und hat keine IDs.
+// Überwachte Dienste: was der Wächter prüft und was auf der Statusseite steht.
+// Vorher war das eine kommagetrennte Einstellung — für eine öffentliche Seite
+// braucht jeder Eintrag einen Namen und die Entscheidung, ob er dort auftaucht.
+db.exec(`
+    CREATE TABLE IF NOT EXISTS monitored_services (
+        id          INTEGER PRIMARY KEY AUTOINCREMENT,
+        name        TEXT NOT NULL,
+        url         TEXT NOT NULL UNIQUE,
+        gruppe      TEXT NOT NULL DEFAULT 'Dienste',
+        oeffentlich INTEGER NOT NULL DEFAULT 0,
+        sort        INTEGER NOT NULL DEFAULT 0
+    );
+`);
+
+// Watchdog: Erreichbarkeit der überwachten Adressen — gleiche Bauart wie beim
+// Game-Server, damit beide denselben Balken benutzen können. Schlüssel bleibt
+// die Adresse und nicht die Dienst-ID: so überlebt der Verlauf ein Umbenennen.
 db.exec(`
     CREATE TABLE IF NOT EXISTS watchdog_history (
         url TEXT NOT NULL,
@@ -723,6 +737,15 @@ db.exec(`
         ms  INTEGER
     );
     CREATE INDEX IF NOT EXISTS idx_wd_url_ts ON watchdog_history (url, ts);
+
+    CREATE TABLE IF NOT EXISTS incidents (
+        id       INTEGER PRIMARY KEY AUTOINCREMENT,
+        url      TEXT NOT NULL,
+        begonnen TEXT NOT NULL DEFAULT (datetime('now')),
+        beendet  TEXT,
+        grund    TEXT
+    );
+    CREATE INDEX IF NOT EXISTS idx_incidents_url ON incidents (url, begonnen DESC);
 `);
 const insertWatchdogSample = db.prepare('INSERT INTO watchdog_history (url, ok, ms) VALUES (?, ?, ?)');
 const watchdogBucketsStmt = db.prepare(`
@@ -734,13 +757,79 @@ const watchdogBucketsStmt = db.prepare(`
     WHERE url = ? AND ts >= datetime('now', ?)
     GROUP BY stunde ORDER BY stunde
 `);
-// Adressen, die aus der Einstellung entfernt wurden, sollen nicht ewig bleiben
-const pruneWatchdogStmt = db.prepare(`DELETE FROM watchdog_history WHERE ts < datetime('now', '-7 days')`);
+// Tagesweise fürs 90-Tage-Bild: stündlich wären das über 2000 Striche, die auf
+// keinen Schirm passen. Statuspage macht es genauso — 90 Tage, ein Strich je Tag.
+const watchdogDaysStmt = db.prepare(`
+    SELECT date(ts)               AS tag,
+           COUNT(*)               AS proben,
+           SUM(ok)                AS an,
+           CAST(AVG(ms) AS INTEGER) AS schnitt
+    FROM watchdog_history
+    WHERE url = ? AND ts >= date('now', ?)
+    GROUP BY tag ORDER BY tag
+`);
+const pruneWatchdogStmt = db.prepare(`DELETE FROM watchdog_history WHERE ts < datetime('now', '-90 days')`);
 export const recordWatchdogSample = (url, ok, ms) =>
     insertWatchdogSample.run(url, ok ? 1 : 0, Number.isFinite(ms) ? Math.round(ms) : null);
 export const watchdogBuckets = (url, hours = 168) => watchdogBucketsStmt.all(url, `-${hours} hours`);
+export const watchdogDays = (url, days = 90) => watchdogDaysStmt.all(url, `-${days} days`);
 export const pruneWatchdogHistory = () => pruneWatchdogStmt.run();
 
+/* ── Störungen: pro Adresse ein offener Eintrag, bis es wieder läuft ── */
+const openIncidentStmt = db.prepare('SELECT * FROM incidents WHERE url = ? AND beendet IS NULL');
+const startIncidentStmt = db.prepare('INSERT INTO incidents (url, grund) VALUES (?, ?)');
+const endIncidentStmt = db.prepare(`UPDATE incidents SET beendet = datetime('now') WHERE id = ?`);
+const listIncidentsStmt = db.prepare(`
+    SELECT id, url, begonnen, beendet, grund,
+           CAST((julianday(COALESCE(beendet, 'now')) - julianday(begonnen)) * 1440 AS INTEGER) AS minuten
+    FROM incidents
+    WHERE begonnen >= date('now', ?)
+    ORDER BY begonnen DESC LIMIT ?
+`);
+const pruneIncidentsStmt = db.prepare(`DELETE FROM incidents WHERE beendet IS NOT NULL AND begonnen < date('now', '-90 days')`);
+
+/** Störung eröffnen — doppelte für dieselbe Adresse verhindert der offene Eintrag */
+export function startIncident(url, grund) {
+    if (openIncidentStmt.get(url)) return false;
+    startIncidentStmt.run(url, grund ?? null);
+    return true;
+}
+export function endIncident(url) {
+    const offen = openIncidentStmt.get(url);
+    if (!offen) return null;
+    endIncidentStmt.run(offen.id);
+    return offen;
+}
+export const listIncidents = (days = 90, limit = 50) => listIncidentsStmt.all(`-${days} days`, limit);
+export const pruneIncidents = () => pruneIncidentsStmt.run();
+
+/* ── Überwachte Dienste ── */
+const listServicesMonStmt = db.prepare('SELECT * FROM monitored_services ORDER BY sort, name');
+const insertServiceMonStmt = db.prepare(`
+    INSERT INTO monitored_services (name, url, gruppe, oeffentlich, sort)
+    VALUES (@name, @url, @gruppe, @oeffentlich, @sort)
+    ON CONFLICT(url) DO NOTHING
+`);
+const updateServiceMonStmt = db.prepare(`
+    UPDATE monitored_services SET name = @name, url = @url, gruppe = @gruppe,
+        oeffentlich = @oeffentlich, sort = @sort WHERE id = @id
+`);
+const deleteServiceMonStmt = db.prepare('DELETE FROM monitored_services WHERE id = ?');
+export const listMonitored = () => listServicesMonStmt.all().map((r) => ({ ...r, oeffentlich: r.oeffentlich === 1 }));
+export const createMonitored = (s) => insertServiceMonStmt.run({
+    name: String(s.name || s.url || '').slice(0, 60),
+    url: s.url,
+    gruppe: s.gruppe || 'Dienste',
+    oeffentlich: s.oeffentlich ? 1 : 0,
+    sort: Number(s.sort) || 0,
+}).lastInsertRowid;
+export const updateMonitored = (s) => updateServiceMonStmt.run({ ...s, oeffentlich: s.oeffentlich ? 1 : 0 });
+export const deleteMonitored = (id) => deleteServiceMonStmt.run(id).changes > 0;
+
+// Die einmalige Übernahme der alten Adressliste steht am Dateiende — sie liest
+// eine Einstellung, und getSetting() hängt an einem const, das erst weiter
+// unten entsteht. Hier aufgerufen stirbt der Bot beim Start.
+
 // Moderation: Verwarnungen + Sticky-Roles
 db.exec(`
     CREATE TABLE IF NOT EXISTS warns (
@@ -985,6 +1074,27 @@ export function setSetting(key, value) {
     setSettingStmt.run(key, value);
 }
 
+// Einmalige Übernahme der alten kommagetrennten Watchdog-Liste in die
+// Dienst-Tabelle. Muss hier unten stehen, weil sie getSetting() braucht.
+//
+// Bewusst nicht öffentlich: was auf der Statusseite auftaucht, soll eine
+// bewusste Entscheidung sein und nicht durch eine Migration passieren.
+{
+    const schonDa = db.prepare('SELECT count(*) AS n FROM monitored_services').get().n;
+    const alt = getSetting('watchdog_urls') ?? '';
+    if (schonDa === 0 && alt.trim()) {
+        const urls = alt.split(',').map((s) => s.trim()).filter(Boolean);
+        for (const [i, url] of urls.entries()) {
+            let name = url;
+            try { name = new URL(url).hostname; } catch { /* bleibt die rohe Adresse */ }
+            createMonitored({ name, url, gruppe: 'Dienste', oeffentlich: false, sort: i });
+        }
+        if (urls.length > 0) {
+            console.log(`[db] ${urls.length} Watchdog-Adresse(n) in die Dienst-Tabelle übernommen`);
+        }
+    }
+}
+
 const insertCommit = db.prepare(`
     INSERT OR IGNORE INTO commits (sha, repo, branch, message, author_name, author_user, url, committed_at)
     VALUES (@sha, @repo, @branch, @message, @author_name, @author_user, @url, @committed_at)

src/web/api.js

@@ -15,7 +15,9 @@ import {
     saveWebAdmin, webAdminScopes, listWebAdmins, deleteWebAdmin,
     saveTemplate, listTemplates, deleteTemplate,
     getLevelRow, isPlaytester, saveWish, toggleWishVote, myWishVotes,
-    logAudit, listAudit, playerHistory, uptimeBuckets, watchdogBuckets, deleteUserData,
+    logAudit, listAudit, playerHistory, uptimeBuckets, watchdogBuckets, watchdogDays,
+    listIncidents, listMonitored, createMonitored, updateMonitored, deleteMonitored,
+    deleteUserData,
     saveSsoApp, listSsoApps, deleteSsoApp, getSsoApp,
     createService, updateService, listServices, deleteService,
     savePage, getPage, listPages, listPublishedPages, deletePage,
@@ -25,7 +27,7 @@ import { moduleStates, moduleDetail, setModuleEnabled, moduleEnabled, moduleFavo
 import { templateStates, setTemplate, TEMPLATE_GROUPS } from '../templates.js';
 import { tuningStates, setTuning, tuning, TUNING_GROUPS } from '../tuning.js';
 import { lastResults, GAME_ICONS } from '../bot/server-monitor.js';
-import { watchdogState, watchedUrls } from '../bot/watchdog.js';
+import { watchdogState, watchedServices } from '../bot/watchdog.js';
 import { publishAppForm, unpublishAppForm, MAX_QUESTIONS } from '../bot/app-forms.js';
 import { sanitizeEmbed } from './api-v1.js';
 import { publishRoleMenu, unpublishRoleMenu, MAX_ENTRIES } from '../bot/role-menus.js';
@@ -190,32 +192,107 @@ export function registerApiRoutes(app, client) {
         }),
     }));
 
-    // Watchdog-Übersicht: Stand und Erreichbarkeit der überwachten Adressen.
-    // Nicht öffentlich — in der Liste können interne Dienste stehen, die
-    // niemand von außen kennen muss.
+    /** Einen überwachten Dienst in die Form bringen, die beide Seiten nutzen */
+    function dienstStand(dienst, { tage = false } = {}) {
+        const s = watchdogState().get(dienst.url) ?? {};
+        return {
+            id: dienst.id,
+            name: dienst.name,
+            gruppe: dienst.gruppe,
+            oeffentlich: dienst.oeffentlich,
+            // undefined = seit dem Start noch nicht geprüft
+            online: s.geprueft ? !s.down : null,
+            fails: s.fails ?? 0,
+            seit: s.since ?? null,
+            ms: s.ms ?? null,
+            status: s.status ?? null,
+            geprueft: s.geprueft ?? null,
+            uptime: tage ? watchdogDays(dienst.url, 90) : watchdogBuckets(dienst.url, 168),
+        };
+    }
+
+    // Watchdog-Übersicht fürs Panel: alle Dienste mit voller Adresse.
+    // Nicht öffentlich — in der Liste können interne Dienste stehen.
     app.get('/api/watchdog', async (request, reply) => {
         if (requireAnyScope(request, reply)) return;
-        const stand = watchdogState();
         return {
             aktiv: moduleEnabled('watchdog'),
             intervall: tuning('watchdog_interval'),
-            adressen: watchedUrls().map((url) => {
-                const s = stand.get(url) ?? {};
+            adressen: watchedServices().map((d) => ({ ...dienstStand(d), url: d.url })),
+            stoerungen: listIncidents(90, 50),
+        };
+    });
+
+    // Öffentliche Statusseite: nur freigegebene Dienste, und bewusst OHNE
+    // Adresse — der Name reicht, die URL geht niemanden etwas an.
+    app.get('/api/status', async () => {
+        const dienste = watchedServices().filter((d) => d.oeffentlich);
+        const urls = new Set(dienste.map((d) => d.url));
+        return {
+            dienste: dienste.map((d) => dienstStand(d, { tage: true })),
+            server: listGameservers().map((s) => {
+                const r = lastResults.get(s.id);
                 return {
-                    url,
-                    // undefined = seit dem Start noch nicht geprüft
-                    online: s.geprueft ? !s.down : null,
-                    fails: s.fails ?? 0,
-                    seit: s.since ?? null,
-                    ms: s.ms ?? null,
-                    status: s.status ?? null,
-                    geprueft: s.geprueft ?? null,
-                    uptime: watchdogBuckets(url, 168),
+                    id: s.id,
+                    name: s.name,
+                    icon: GAME_ICONS[s.type] ?? '🎮',
+                    online: r?.online ?? null,
+                    players: r?.players ?? null,
+                    max: r?.max ?? null,
                 };
             }),
+            // Nur Störungen freigegebener Dienste — sonst verrät die Historie,
+            // was es sonst noch gibt
+            stoerungen: listIncidents(90, 20)
+                .filter((v) => urls.has(v.url))
+                .map(({ url, ...rest }) => ({
+                    ...rest,
+                    name: dienste.find((d) => d.url === url)?.name ?? 'Dienst',
+                })),
         };
     });
 
+    // --- Überwachte Dienste verwalten (Panel) ---
+
+    /** Eingaben prüfen; antwortet bei Fehlern selbst */
+    function dienstBody(request, reply) {
+        const b = request.body ?? {};
+        const name = String(b.name ?? '').trim().slice(0, 60);
+        const url = String(b.url ?? '').trim();
+        if (!name) { reply.code(400).send({ error: 'Name fehlt' }); return null; }
+        if (!/^https?:\/\/.+/.test(url)) { reply.code(400).send({ error: 'URL mit http(s) erwartet' }); return null; }
+        return {
+            name, url,
+            gruppe: String(b.gruppe ?? 'Dienste').trim().slice(0, 40) || 'Dienste',
+            oeffentlich: Boolean(b.oeffentlich),
+            sort: Number(b.sort) || 0,
+        };
+    }
+
+    app.post('/api/monitored', async (request, reply) => {
+        if (requireScope(request, reply, 'settings')) return;
+        const d = dienstBody(request, reply);
+        if (!d) return;
+        createMonitored(d);
+        logAudit(getSessionUser(request), 'dienst überwacht', `${d.name} (${d.url})`);
+        return { ok: true, dienste: listMonitored() };
+    });
+
+    app.put('/api/monitored/:id', async (request, reply) => {
+        if (requireScope(request, reply, 'settings')) return;
+        const d = dienstBody(request, reply);
+        if (!d) return;
+        updateMonitored({ ...d, id: Number(request.params.id) });
+        logAudit(getSessionUser(request), 'dienst geändert', d.name);
+        return { ok: true, dienste: listMonitored() };
+    });
+
+    app.delete('/api/monitored/:id', async (request, reply) => {
+        if (requireScope(request, reply, 'settings')) return;
+        logAudit(getSessionUser(request), 'dienst entfernt', String(request.params.id));
+        return { deleted: deleteMonitored(Number(request.params.id)), dienste: listMonitored() };
+    });
+
     // --- Module: zentrale An/Aus-Schalter aller Funktionen ---
 
     app.get('/api/modules', async (request, reply) => {