Offene Lobbys nach 3 Stunden samt Server löschen (LOBBY_MAX_HOURS)

Der Aufräum-Lauf löscht Lobbys, deren Veto nie gestartet wurde, nach LOBBY_MAX_HOURS (Standard 3, 0 = nie) seit dem Erstellen und stoppt ihren Server. Matches im Veto, im Spiel oder abgeschlossen bleiben unberührt.

server/src/config.js

@@ -105,6 +105,8 @@ export function loadConfig(overrides = {}) {
     serverAdapter: str('SERVER_ADAPTER', 'demo'),
     idleTimeoutMinutes: int('SERVER_IDLE_TIMEOUT_MINUTES', 30),
     maxLifetimeHours: int('SERVER_MAX_LIFETIME_HOURS', 8),
+    // Offene Lobbys (Veto nie gestartet) werden nach so vielen Stunden gelöscht. 0 = nie
+    lobbyMaxHours: int('LOBBY_MAX_HOURS', 3),
     statusPollSeconds: int('SERVER_STATUS_POLL_SECONDS', 5),
     reaperIntervalSeconds: int('SERVER_REAPER_INTERVAL_SECONDS', 60),
     // Frist pro Veto-Schritt in Sekunden, danach führt das Backend den Schritt automatisch aus. 0 = keine Frist

server/src/servers/serverManager.js

@@ -132,12 +132,13 @@ export class ServerManager {
   }
 
   /** Löscht ein Match samt Server-Datensatz. Ein laufender Server wird vorher freigegeben. */
-  async deleteMatch(matchId) {
+  async deleteMatch(matchId, reason = 'Match vom Admin gelöscht') {
     this.service.getMatch(matchId);
     clearTimeout(this.autoAdvance.get(matchId));
     this.autoAdvance.delete(matchId);
     this.simulator.stop(matchId);
-    await this.stop(matchId, 'Match vom Admin gelöscht');
+    this.knifeAttempts.delete(matchId);
+    await this.stop(matchId, reason);
     this.db.prepare('DELETE FROM matches WHERE id = ?').run(matchId);
     this.service.emit('changed', matchId);
   }
@@ -687,8 +688,22 @@ export class ServerManager {
 
   // ---------- Automatisches Herunterfahren ----------
 
+  /** Lobbys, die länger als LOBBY_MAX_HOURS offen sind (Veto nie gestartet), samt Server löschen. */
+  async deleteStaleLobbies(now = Date.now()) {
+    const hours = this.config.lobbyMaxHours;
+    if (!hours) return 0;
+    const stale = this.db.prepare("SELECT id FROM matches WHERE status = 'lobby' AND created_at < ?").all(now - hours * 3_600_000);
+    for (const { id } of stale) {
+      if (this.busy.has(id)) continue;
+      this.logger.log?.(`[aufräumen] Lobby ${id} gelöscht (länger als ${hours} h offen)`);
+      await this.deleteMatch(id, `Lobby länger als ${hours} h offen`).catch((err) => this.logger.error(err));
+    }
+    return stale.length;
+  }
+
   async reap(now = Date.now()) {
     this.service.archiveClosed(now);
+    await this.deleteStaleLobbies(now);
     const timeoutMs = this.config.idleTimeoutMinutes * 60_000;
     const rows = this.db.prepare(`SELECT * FROM servers WHERE status IN (${ACTIVE.map(() => '?').join(',')})`).all(...ACTIVE);
     for (const row of rows) {