FDS2610/monitor
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request 'Update-Sicherheit: Tests vor Dienst-Neustart, API-Banner uebersteht Render-Fehler' (#95) from fds2610-bot/monitor:main into main

Browse source 
Reviewed-on: https://codeberg.org/fds2610/monitor/pulls/95
This commit is contained in:
fds2610 2026-09-23 08:28:44 +02:00
commit 18990c2d80
2 changed files with 19 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
html/js/office-render.js
View file

@ -2687,8 +2687,16 @@ async function boot() {
setApiBanner(false);
} catch (err) {
console.warn("[office] Live-API nicht erreichbar, zeige Demo-Daten.", err);
renderOfficeScene(container);
setApiBanner(true);
// setApiBanner() steht in einem eigenen finally: renderOfficeScene() zeichnet bei jedem
// Refresh die komplette Szene neu und kann durch einen Fehler in noch unbekannten
// Datenkombinationen scheitern (siehe Issue #94 - Produktion zeigte trotz nicht
// erreichbarer API keinen Banner). Ohne das finally wuerde ein solcher Fehler die
// wichtigere Meldung "API nicht erreichbar" mit sich reissen, statt sie anzuzeigen.
try {
renderOfficeScene(container);
} finally {
setApiBanner(true);
}
}
}

12
update.sh
View file

@ -78,9 +78,12 @@ $DOCKER_COMPOSE run --rm api composer install --no-interaction --no-progress
log "Baue Docker-Images neu (falls Dockerfile/Abhaengigkeiten sich geaendert haben)..."
$DOCKER_COMPOSE build
log "Starte Dienste neu (api, web, sync)..."
$DOCKER_COMPOSE up -d api web sync
# Tests laufen bewusst VOR dem Neustart der Dienste, in einem temporaeren Container
# (docker compose run --rm), der die produktiven api/web/sync-Container nicht anfasst. Liefe
# "up -d" zuerst, wuerden fehlschlagende Tests das Skript zwar per "set -e" abbrechen, die
# Produktion liefe aber bereits mit dem fehlerhaften Stand weiter - und deploy-webhook.py
# meldet einen solchen Abbruch nirgends aktiv, nur still im Log. Bei einem Fehlschlag hier
# bleiben die alten Container unveraendert im Einsatz.
if [ "$UPDATE_RUN_TESTS" = "1" ]; then
log "Fuehre PHPUnit-Tests (API) aus..."
$DOCKER_COMPOSE run --rm api composer test
@ -91,4 +94,7 @@ else
log "UPDATE_RUN_TESTS=0 gesetzt - ueberspringe Testsuiten."
fi
log "Starte Dienste neu (api, web, sync)..."
$DOCKER_COMPOSE up -d api web sync
log "Update abgeschlossen (${UPDATE_BRANCH} @ $(git rev-parse --short HEAD))."