httpclient, tenantregistryclient: Quelltext identisch zu common (normalisierter Vergleich), interne Kopien geloescht.
apierror: dms-Fassung war die alte (5xx-Text sichtbar). common/apierror enthaelt die Behebung aus #375 (Text Interner Fehler, Korrelations-ID, Log); die Tests dazu liegen in common/apierror/apierror_test.go (5xx verbirgt Handlertext, IDs eindeutig). Damit entfaellt #375.
Waechtertest gegen http.DefaultClient/http.Get(...) zog nach dms/internal/archtest/defaultclient_test.go.
Auswirkungen (Pflicht-Review)
API-Verhalten: 5xx-Antworten von dms zeigen kuenftig Interner Fehler plus correlation_id statt des Handlertexts (gewollt, QA-15 F-14). Clients, die den 5xx-Text auswerten, muessten angepasst werden.
dms braucht jetzt GOPRIVATE=gitea.perlbach24.de und Lesezugriff aufs Repo bei Einzelbuilds.
Nicht umgestellt: httpharden, policyclient (dms-Varianten), migrate (wartet auf common v0.2.0 mit Tabellenname als Parameter), testdb (eigener PR).
Tests
131, frisches Wurfverzeichnis: go build/vet ./..., go test ./... -count=1 ohne Fehler, gofmt sauber. DB-Tests nicht gelaufen (kein TEST_ADMIN_DSN); Lauf auf 204 gegen dms_dev steht aus.
Umstellung 2 (nach archive-Pilot #383). Ersetzt zugleich #375.
## Umgestellt
- `httpclient`, `tenantregistryclient`: Quelltext identisch zu common (normalisierter Vergleich), interne Kopien geloescht.
- `apierror`: dms-Fassung war die alte (5xx-Text sichtbar). `common/apierror` enthaelt die Behebung aus #375 (Text `Interner Fehler`, Korrelations-ID, Log); die Tests dazu liegen in `common/apierror/apierror_test.go` (5xx verbirgt Handlertext, IDs eindeutig). Damit entfaellt #375.
- Waechtertest gegen `http.DefaultClient`/`http.Get(...)` zog nach `dms/internal/archtest/defaultclient_test.go`.
## Auswirkungen (Pflicht-Review)
- API-Verhalten: 5xx-Antworten von dms zeigen kuenftig `Interner Fehler` plus `correlation_id` statt des Handlertexts (gewollt, QA-15 F-14). Clients, die den 5xx-Text auswerten, muessten angepasst werden.
- dms braucht jetzt `GOPRIVATE=gitea.perlbach24.de` und Lesezugriff aufs Repo bei Einzelbuilds.
- Nicht umgestellt: `httpharden`, `policyclient` (dms-Varianten), `migrate` (wartet auf common v0.2.0 mit Tabellenname als Parameter), `testdb` (eigener PR).
## Tests
131, frisches Wurfverzeichnis: `go build/vet ./...`, `go test ./... -count=1` ohne Fehler, gofmt sauber. **DB-Tests nicht gelaufen** (kein `TEST_ADMIN_DSN`); Lauf auf 204 gegen `dms_dev` steht aus.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Die common-Fassung von apierror enthaelt die 5xx-Behebung (Text verborgen,
Korrelations-ID). Der Waechtertest gegen http.DefaultClient liegt jetzt in
internal/archtest.
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
DB-Tests 2026-10-28 auf 204 gegen einen eigenen Wegwerf-Postgres 17 (initdb in /tmp, Port 55432, getrennte Datenbanken fuer TEST_ADMIN_DSN, TEST_TENANT_DSN und TEST_TENANT_DSN_B; nicht archivdms, nicht dms_dev), go test ./... -count=1 -p 1. Stand refactor/dms-testdb (enthaelt diesen PR). Drei Fehler, alle identisch auf main (Baseline, nicht durch diesen PR): audittrail TestAppendOnly_AppRoleHasNoUpdateOrDeletePrivilege (audit_log fehlt im Testschema), qa03 TestReferenceCorpus_FullLifecycle und qa06 TestActiveContent_NeverServedInline (Upload-Typpruefung lehnt Testdateien als application/zip, text/xml bzw. text/html ab). Alle anderen Pakete gruen.
DB-Tests 2026-10-28 auf 204 gegen einen eigenen Wegwerf-Postgres 17 (initdb in /tmp, Port 55432, getrennte Datenbanken fuer TEST_ADMIN_DSN, TEST_TENANT_DSN und TEST_TENANT_DSN_B; nicht archivdms, nicht dms_dev), go test ./... -count=1 -p 1. Stand refactor/dms-testdb (enthaelt diesen PR). Drei Fehler, alle identisch auf main (Baseline, nicht durch diesen PR): audittrail TestAppendOnly_AppRoleHasNoUpdateOrDeletePrivilege (audit_log fehlt im Testschema), qa03 TestReferenceCorpus_FullLifecycle und qa06 TestActiveContent_NeverServedInline (Upload-Typpruefung lehnt Testdateien als application/zip, text/xml bzw. text/html ab). Alle anderen Pakete gruen.
patrick
merged commit fbb41235c0 into main2026-10-28 10:41:20 +01:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Umstellung 2 (nach archive-Pilot #383). Ersetzt zugleich #375.
Umgestellt
httpclient,tenantregistryclient: Quelltext identisch zu common (normalisierter Vergleich), interne Kopien geloescht.apierror: dms-Fassung war die alte (5xx-Text sichtbar).common/apierrorenthaelt die Behebung aus #375 (TextInterner Fehler, Korrelations-ID, Log); die Tests dazu liegen incommon/apierror/apierror_test.go(5xx verbirgt Handlertext, IDs eindeutig). Damit entfaellt #375.http.DefaultClient/http.Get(...)zog nachdms/internal/archtest/defaultclient_test.go.Auswirkungen (Pflicht-Review)
Interner Fehlerpluscorrelation_idstatt des Handlertexts (gewollt, QA-15 F-14). Clients, die den 5xx-Text auswerten, muessten angepasst werden.GOPRIVATE=gitea.perlbach24.deund Lesezugriff aufs Repo bei Einzelbuilds.httpharden,policyclient(dms-Varianten),migrate(wartet auf common v0.2.0 mit Tabellenname als Parameter),testdb(eigener PR).Tests
131, frisches Wurfverzeichnis:
go build/vet ./...,go test ./... -count=1ohne Fehler, gofmt sauber. DB-Tests nicht gelaufen (keinTEST_ADMIN_DSN); Lauf auf 204 gegendms_devsteht aus.🤖 Generated with Claude Code
DB-Tests 2026-10-28 auf 204 gegen einen eigenen Wegwerf-Postgres 17 (initdb in /tmp, Port 55432, getrennte Datenbanken fuer TEST_ADMIN_DSN, TEST_TENANT_DSN und TEST_TENANT_DSN_B; nicht archivdms, nicht dms_dev), go test ./... -count=1 -p 1. Stand refactor/dms-testdb (enthaelt diesen PR). Drei Fehler, alle identisch auf main (Baseline, nicht durch diesen PR): audittrail TestAppendOnly_AppRoleHasNoUpdateOrDeletePrivilege (audit_log fehlt im Testschema), qa03 TestReferenceCorpus_FullLifecycle und qa06 TestActiveContent_NeverServedInline (Upload-Typpruefung lehnt Testdateien als application/zip, text/xml bzw. text/html ab). Alle anderen Pakete gruen.