nexarch/docs/API-23-PRUEFPROTOKOLL.md

API-23 – Prüfprotokoll: HTTP-Client mit Zeitlimit im Core-Root-Modul

Nachtrag zu API-19 (QA-15-Befund F-11): das Core-Root-Modul selbst war bei der damaligen Umstellung nicht erfasst.

Umsetzung

• internal/httpclient (neu, Core-Root-Modul): identisches Muster wie dms/internal/httpclient, archive/internal/httpclient, mail/internal/httpclient — New(timeout) liefert einen *http.Client mit Zeitlimit und CheckRedirect, das http.ErrUseLastResponse zurückgibt (keine Weiterleitungen); Default() liefert den gemeinsamen Client mit DefaultTimeout = 15s.
• internal/statuspage/statuspage.go: NewHTTPChecker nutzt jetzt httpclient.Default() statt eines ungebundenen &http.Client{}.
• internal/moduletrust/fetch.go: FetchJWKSFromURL nutzt bei client == nil jetzt httpclient.Default() statt http.DefaultClient.
• Architekturtest TestNoUnboundedDefaultClientInProductionCode in internal/httpclient/httpclient_test.go: durchsucht nur internal/ und cmd/ (die eigenständigen Module dms/, archive/, mail/, connect/, ai/, workflow/ liegen zwar im selben Repo, sind aber eigene Go-Module mit eigenem Architekturtest und daher hier bewusst ausgeklammert). Erkennt zusätzlich zu http.DefaultClient/http.Get|Post|Head|PostForm( auch den ungebundenen http.Client{} (genau das Muster, das in statuspage.go gefunden wurde).

Prüfungen

#	Prüfung	Ergebnis
1	Hängender Testserver: Aufruf über internal/statuspage/internal/moduletrust bricht nach ~15s ab statt unbegrenzt zu warten	bestanden – TestClientGivesUpOnHangingServer (via internal/httpclient, das beide Aufrufer jetzt nutzen): httptest-Server hängt endlos, New(200ms).Get(...) liefert nach Zeitlimit einen Fehler statt unbegrenzt zu warten
2	Testserver mit Weiterleitung: Aufruf folgt der Weiterleitung nicht	bestanden – TestClientDoesNotFollowRedirects: Weiterleitungsziel-Handler ruft t.Error auf, falls er je erreicht wird; Default().Do(req) liefert den 307 der Weiterleitungsantwort selbst, das Ziel wird nie aufgerufen
3	Architekturtest schlägt fehl, wenn http.DefaultClient/http.Client{} wieder eingeführt wird (echt provoziert, nicht nur behauptet)	bestanden – real provoziert: internal/moduletrust/fetch.go testweise auf client = http.DefaultClient zurückgesetzt, go test ./internal/httpclient/... -run TestNoUnbounded schlägt mit exakter Datei:Zeile-Angabe fehl; Datei danach wiederhergestellt, go build ./... wieder sauber

Build/Test-Ergebnis (192.168.1.204)

go build ./...                                                        -> clean
go vet ./...                                                          -> clean
go test ./internal/httpclient/... ./internal/statuspage/... \
        ./internal/moduletrust/... -v                                 -> alle PASS
                                                                          (Integrationstests ohne
                                                                          TEST_ADMIN_DSN übersprungen,
                                                                          unverändert durch dieses Ticket)

Gesamtergebnis

Bestanden. Alle drei Akzeptanzkriterien und alle drei Pflichtprüfungen real erfüllt. Damit ist F-11/API-19 nun auch für das Core-Root-Modul nachgezogen; dms, archive, mail bereits durch API-19 abgedeckt.