connect: auditclient, httpclient, policyclient aus common v0.1.0 #388

Merged
patrick merged 1 commits from refactor/connect-common into main 2026-10-28 10:41:40 +01:00
Owner

Umstellung auf common v0.1.0 fuer connect (nach archive #383, dms #384, mail #386).

Umgestellt

auditclient, httpclient, policyclient: Quelltext identisch zu common (normalisierter Vergleich ohne Kommentare/Imports), interne Kopien geloescht, Importe auf gitea.perlbach24.de/scripte/nexarch/common/<paket>, go.mod/go.sum ergaenzt.

Auswirkungen (Pflicht-Review)

  • Kein Verhaltensunterschied (identischer Quelltext).
  • connect braucht jetzt GOPRIVATE=gitea.perlbach24.de und Lesezugriff aufs Repo bei Einzelbuilds.
  • scripts/check-security-test-coverage.sh: Eintraege fuer internal/auditclient und internal/policyclient entfernt (Pakete in common, dort durch das Gate gedeckt).
  • Kein Waechtertest verschoben: connect hatte keinen gegen http.DefaultClient (cmd/datev-a11y-fixture nutzt ihn).
  • Nicht umgestellt: loadtest (Quelltext weicht ab), migrate (wartet auf common v0.2.0), testdb.
  • Testergebnis: alle Pakete ok.

Tests

131, frisches Wurfverzeichnis: build, vet, gofmt sauber; go test ./... siehe unten. DB-Tests nicht gelaufen (kein TEST_ADMIN_DSN), Lauf auf 204 steht aus.

🤖 Generated with Claude Code

Umstellung auf `common` v0.1.0 fuer `connect` (nach archive #383, dms #384, mail #386). ## Umgestellt `auditclient`, `httpclient`, `policyclient`: Quelltext identisch zu common (normalisierter Vergleich ohne Kommentare/Imports), interne Kopien geloescht, Importe auf `gitea.perlbach24.de/scripte/nexarch/common/<paket>`, `go.mod`/`go.sum` ergaenzt. ## Auswirkungen (Pflicht-Review) - Kein Verhaltensunterschied (identischer Quelltext). - connect braucht jetzt `GOPRIVATE=gitea.perlbach24.de` und Lesezugriff aufs Repo bei Einzelbuilds. - `scripts/check-security-test-coverage.sh`: Eintraege fuer `internal/auditclient` und `internal/policyclient` entfernt (Pakete in common, dort durch das Gate gedeckt). - Kein Waechtertest verschoben: connect hatte keinen gegen `http.DefaultClient` (`cmd/datev-a11y-fixture` nutzt ihn). - Nicht umgestellt: `loadtest` (Quelltext weicht ab), `migrate` (wartet auf common v0.2.0), `testdb`. - Testergebnis: alle Pakete ok. ## Tests 131, frisches Wurfverzeichnis: build, vet, gofmt sauber; `go test ./...` siehe unten. DB-Tests nicht gelaufen (kein `TEST_ADMIN_DSN`), Lauf auf 204 steht aus. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
patrick added 1 commit 2026-10-27 23:34:05 +01:00
connect: auditclient, httpclient, policyclient aus common v0.1.0
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (push) Canceled after 0s
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (pull_request) Canceled after 0s
CI / Core-Modul (Build/Vet/Test) (push) Failing after 1m4s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (push) Successful in 4s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (push) Successful in 4s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (push) Failing after 25s
CI / Web (dms-browser) (push) Failing after 1m31s
CI / Web (pdf-viewer) (push) Failing after 1m2s
CI / Web (shl) (push) Failing after 12s
Connect-Schnittstellen-Vertragstests / contract-tests (push) Successful in 10m0s
Supply-Chain-Scan (Go) / govulncheck (push) Failing after 45s
CI / Core-Modul (Build/Vet/Test) (pull_request) Failing after 1m4s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (pull_request) Successful in 3s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (pull_request) Successful in 3s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Failing after 25s
CI / Web (dms-browser) (pull_request) Failing after 1m33s
CI / Web (pdf-viewer) (pull_request) Failing after 1m4s
CI / Web (shl) (pull_request) Failing after 15s
Connect-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 11m18s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 10m25s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Failing after 1m0s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Failing after 2m4s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 3m28s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Successful in 10m55s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (push) Failing after 3s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (pull_request) Failing after 3s
DB-Integrationstests / db-tests (pull_request) Canceled after 0s
7958102870
auditclient und policyclient aus der Liste in check-security-test-coverage.sh
entfernt (Pakete liegen jetzt in common, dort durch das common-Gate gedeckt).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Author
Owner

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/connect-testdb (enthaelt diesen PR): alle 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/connect-testdb (enthaelt diesen PR): alle Pakete gruen.
patrick merged commit 95582ad09d into main 2026-10-28 10:41:40 +01:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: scripte/nexarch#388