fix(abnahme): Healthcheck schweigt, zwei Konventionen praezisiert #41

Merged

florian.wilde merged 2 commits from feat/39-40-nachtrag-abnahme into main 2026-10-04 14:12:29 +00:00

Conversation 0 Commits 2 Files Changed 12

+307 -31

2 Commits

Author	SHA1	Message	Date
Florian Wilde	77389a2c5f	docs(konventionen): Hexwert-Ausnahmen und LDAP-Container festhalten ... Zwei Saetze in CLAUDE.md beschrieben den Code nicht genau. "Nie auf rohe Hexwerte" hat sechs begruendete Ausnahmen, nicht null: Papier, ein QR-Decoder und das Web-App-Manifest entscheiden nicht nach unserem Farbschema. Sie stehen jetzt als Tabelle mit Begruendung je Zeile. Die Regel wurde ergaenzt, nicht der Code geaendert -- eine Konstante ueber drei unverwandte Dinge haette die Begruendung versteckt. Das Tor vor jedem Commit nennt den LDAP-Container. Ohne ihn ueberspringt die Suite 22 Tests -- nachgezaehlt, nicht geschaetzt: 81 mit Container, 59 ohne. Sie sagt es auf stderr, der Lauf sieht trotzdem gruen aus. Im README fehlte zusaetzlich format:pruefen, das CLAUDE.md seit #010 verlangt. Damit das haelt, auch wenn niemand hinsieht: pruefe:doku vergleicht die beiden Schranken-Bloecke und verlangt von beiden den Container. Gegenprobe gefahren. Der Review hatte vier Hexwerte genannt und src/lib/qr.ts uebersehen, und das Issue behauptete die Regel stehe auch in docs/architektur.md -- tut sie nicht. Beides im Issue-Text richtiggestellt, statt einen Haken ueber falschen Text zu setzen. Closes #40	2026-10-04 16:11:45 +02:00
Florian Wilde	2e8681abf5	fix(gesundheit): Fehlergrund nicht mehr unangemeldet herausgeben ... /api/gesundheit steht in der Allowlist des Proxys, damit ein Healthcheck kein Cookie braucht. Eine Meldung von Prisma oder pg nennt aber Host, Port, Benutzernamen und Datenbanknamen -- fuer einen Pfad, der ohne Anmeldung antwortet, ist das zu viel. Nach aussen bleibt {"status":"fehler"} mit 503, damit Docker und der Reverse Proxy weiter richtig entscheiden. Die Begruendung geht ins Protokoll des Prozesses, nach demselben Muster wie in src/server/actions/verzeichnis.ts. Fuenf Tests, darunter einer fuer die andere Haelfte der Zusage: dass die Begruendung wirklich geschrieben und nicht verschluckt wird. Gegenprobe gefahren -- mit der alten Fassung werden drei davon rot. Closes #39	2026-10-04 16:11:28 +02:00