connect: Bearer-Pruefung in connect-app und Frontends #419

Merged
patrick merged 3 commits from feat/connect-bearer into main 2026-10-28 23:30:24 +01:00
Owner

Baut auf #417 (shl-Helfer createBearerClient). Erst #417 mergen, dann zeigt dieser PR nur noch seinen eigenen Commit (3f830382).

Aenderung

  • connect/internal/bearerauth: wie workflow/internal/bearerauth; ein verifiziertes Core-Token setzt X-User-Id/X-Tenant-Slug, Client-Header werden ueberschrieben. Ohne Token werden sie entfernt, ausser NEXARCH_CONNECT_APP_ALLOW_HEADER_AUTH=1 (Uebergang).
  • connect-app: NEXARCH_CONNECT_APP_JWKS_URL, _JWT_AUDIENCE (Default connect), _JWT_ISSUER, _JWT_AUDIENCE_ENFORCE. Ohne JWKS-URL und ohne Uebergangsschalter startet der Dienst nicht. Mit JWKS ist NEXARCH_CONNECT_APP_TENANT_SLUG Pflicht, weil die Datenbank genau einem Mandanten gehoert.
  • connect/go.mod: common v0.2.0 auf v0.2.1 (moduleauth).
  • Frontends connect-setup, connect-status, datev-export: lib/http.ts (createBearerClient("connect")), Aufrufe ueber apiFetch, keine Identitaets-Header mit Bearer.
  • dms2-import: BFF-Routen reichen mit Bearer nur das Token weiter; Seite nimmt Nutzer und Mandant aus der Sitzung; Tests bearer.route.test.ts.

Auswirkung

  • Breaking beim Deploy: connect-app startet nur noch mit NEXARCH_CONNECT_APP_JWKS_URL (plus _TENANT_SLUG) oder mit NEXARCH_CONNECT_APP_ALLOW_HEADER_AUTH=1. Bestandsinstallationen muessen eine der beiden Varianten setzen, sonst Startfehler.
  • Core braucht connect in NEXARCH_ACCOUNT_MODULE_AUDIENCES; die Frontends fordern ?module=connect an, sobald NEXT_PUBLIC_CORE_API_BASE gesetzt ist (sonst Altpfad mit Umgebungsvariablen).
  • Direkter Browser-Aufruf der Connect-API aus connect-setup/-status/datev-export braucht CORS fuer Authorization (auf 204 ueber nginx 8447/8448 geloest, auf 131 pruefen).
  • datevapi ist in connect-app nicht registriert (nur im a11y-Fixture); die Middleware gilt fuer alles auf dem Mux.
  • bearerauth ist eine Kopie des workflow-Pakets; Verschieben nach common/ waere ein Folge-PR (neuer common-Tag).
  • Beruehrt dieselben Dateien wie #418 (connect/cmd/connect-app/main.go): Konflikt moeglich, Rebase auf den zuerst gemergten.

Tests (131)

connect: bearerauth, statusapi, connectorsetupapi, dms2importapi, datevapi ok, go vet/gofmt sauber. Frontends: connect-setup 4, connect-status 5, datev-export 4, dms2-import 11; tsc sauber; next build --webpack von dms2-import ok.

🤖 Generated with Claude Code

**Baut auf #417** (shl-Helfer `createBearerClient`). Erst #417 mergen, dann zeigt dieser PR nur noch seinen eigenen Commit (3f830382). ## Aenderung - `connect/internal/bearerauth`: wie `workflow/internal/bearerauth`; ein verifiziertes Core-Token setzt `X-User-Id`/`X-Tenant-Slug`, Client-Header werden ueberschrieben. Ohne Token werden sie entfernt, ausser `NEXARCH_CONNECT_APP_ALLOW_HEADER_AUTH=1` (Uebergang). - `connect-app`: `NEXARCH_CONNECT_APP_JWKS_URL`, `_JWT_AUDIENCE` (Default `connect`), `_JWT_ISSUER`, `_JWT_AUDIENCE_ENFORCE`. Ohne JWKS-URL und ohne Uebergangsschalter startet der Dienst nicht. Mit JWKS ist `NEXARCH_CONNECT_APP_TENANT_SLUG` Pflicht, weil die Datenbank genau einem Mandanten gehoert. - `connect/go.mod`: common v0.2.0 auf v0.2.1 (moduleauth). - Frontends `connect-setup`, `connect-status`, `datev-export`: `lib/http.ts` (`createBearerClient("connect")`), Aufrufe ueber `apiFetch`, keine Identitaets-Header mit Bearer. - `dms2-import`: BFF-Routen reichen mit Bearer nur das Token weiter; Seite nimmt Nutzer und Mandant aus der Sitzung; Tests `bearer.route.test.ts`. ## Auswirkung - **Breaking beim Deploy:** `connect-app` startet nur noch mit `NEXARCH_CONNECT_APP_JWKS_URL` (plus `_TENANT_SLUG`) oder mit `NEXARCH_CONNECT_APP_ALLOW_HEADER_AUTH=1`. Bestandsinstallationen muessen eine der beiden Varianten setzen, sonst Startfehler. - Core braucht `connect` in `NEXARCH_ACCOUNT_MODULE_AUDIENCES`; die Frontends fordern `?module=connect` an, sobald `NEXT_PUBLIC_CORE_API_BASE` gesetzt ist (sonst Altpfad mit Umgebungsvariablen). - Direkter Browser-Aufruf der Connect-API aus `connect-setup/-status`/`datev-export` braucht CORS fuer `Authorization` (auf 204 ueber nginx 8447/8448 geloest, auf 131 pruefen). - `datevapi` ist in `connect-app` nicht registriert (nur im a11y-Fixture); die Middleware gilt fuer alles auf dem Mux. - `bearerauth` ist eine Kopie des workflow-Pakets; Verschieben nach `common/` waere ein Folge-PR (neuer common-Tag). - Beruehrt dieselben Dateien wie #418 (`connect/cmd/connect-app/main.go`): Konflikt moeglich, Rebase auf den zuerst gemergten. ## Tests (131) connect: `bearerauth`, `statusapi`, `connectorsetupapi`, `dms2importapi`, `datevapi` ok, `go vet`/`gofmt` sauber. Frontends: connect-setup 4, connect-status 5, datev-export 4, dms2-import 11; `tsc` sauber; `next build --webpack` von dms2-import ok. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
patrick added 2 commits 2026-10-28 22:48:20 +01:00
shl: createBearerClient und bearerHeader statt vier kopierter http/forward-Dateien
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (push) Canceled after 0s
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (pull_request) Canceled after 0s
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (push) Canceled after 0s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (push) Successful in 3s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (push) Successful in 3s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (push) Failing after 13s
CI / Core-Modul (Build/Vet/Test) (pull_request) Successful in 36s
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 2s
CI / Core-Modul (Build/Vet/Test) (push) Successful in 40s
CI / Web (dms-browser) (push) Failing after 53s
CI / Web (pdf-viewer) (push) Failing after 40s
CI / Web (shl) (push) Failing after 8s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Failing after 12s
CI / Web (dms-browser) (pull_request) Failing after 47s
CI / Web (pdf-viewer) (pull_request) Failing after 40s
CI / Web (shl) (pull_request) Failing after 9s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 15m45s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Failing after 51s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Failing after 1m16s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 15m22s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Successful in 15m39s
e2d941287f
workflow-approvals, workflow-tasks, workflow-cases und routing-rules hatten
identische lib/http.ts und lib/forward.ts; mail-search eine fuenfte Variante.
Verhalten unveraendert.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
connect: Bearer-Pruefung in connect-app, Frontends connect-*, datev-export, dms2-import per Sitzung
DB-Integrationstests / db-tests (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (push) Canceled after 0s
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) Successful in 40s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (push) Successful in 2s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (push) Successful in 3s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (push) Failing after 14s
Supply-Chain-Scan (Go) / govulncheck (push) Failing after 42s
CI / Core-Modul (Build/Vet/Test) (pull_request) Successful in 41s
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 2s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Failing after 16s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 22m20s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Failing after 1m3s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Failing after 1m54s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Successful in 23m56s
CI / Web (dms-browser) (push) Failing after 49s
CI / Web (pdf-viewer) (push) Failing after 37s
CI / Web (shl) (push) Failing after 8s
CI / Web (dms-browser) (pull_request) Failing after 55s
CI / Web (pdf-viewer) (pull_request) Failing after 38s
CI / Web (shl) (pull_request) Failing after 8s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 23m46s
3f83038203
connect/internal/bearerauth (wie workflow): verifiziertes Core-Token setzt
X-User-Id/X-Tenant-Slug, ohne Token nur mit NEXARCH_CONNECT_APP_ALLOW_HEADER_AUTH=1.
Mit JWKS ist NEXARCH_CONNECT_APP_TENANT_SLUG Pflicht (Mandant der Datenbank).
common auf v0.2.1 (moduleauth). Frontends nutzen createBearerClient("connect").

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
patrick added 1 commit 2026-10-28 23:09:20 +01:00
connect bearerauth: X-User-Role nur aus dem Token
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) Successful in 1m16s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (push) Successful in 5s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (push) Successful in 4s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (push) Failing after 21s
CI / Web (dms-browser) (push) Failing after 1m27s
CI / Web (pdf-viewer) (push) Failing after 1m3s
CI / Web (shl) (push) Failing after 11s
CI / Core-Modul (Build/Vet/Test) (pull_request) Successful in 1m4s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (pull_request) Successful in 4s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (pull_request) Successful in 4s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Failing after 22s
CI / Web (dms-browser) (pull_request) Failing after 1m39s
CI / Web (pdf-viewer) (pull_request) Failing after 59s
CI / Web (shl) (pull_request) Failing after 12s
Supply-Chain-Scan (Go) / govulncheck (push) Failing after 1m1s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 24m39s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Failing after 52s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Failing after 1m51s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 25m50s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Successful in 24m33s
DB-Integrationstests / db-tests (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (push) Canceled after 0s
bfbb375649
Gleiche Aenderung wie workflow/internal/bearerauth, damit die Rechtepruefung
aus #418 die echte Rolle bekommt.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
patrick added 1 commit 2026-10-28 23:30:21 +01:00
Merge main in feat/connect-bearer
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) Successful in 1m0s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (push) Successful in 3s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (push) Successful in 4s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (push) Failing after 28s
CI / Web (dms-browser) (push) Failing after 1m27s
CI / Web (pdf-viewer) (push) Failing after 1m13s
CI / Web (shl) (push) Failing after 17s
CI / Core-Modul (Build/Vet/Test) (pull_request) Successful in 1m3s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (pull_request) Successful in 4s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (pull_request) Successful in 4s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Failing after 18s
CI / Web (dms-browser) (pull_request) Failing after 1m38s
CI / Web (pdf-viewer) (pull_request) Failing after 1m6s
CI / Web (shl) (pull_request) Failing after 11s
Supply-Chain-Scan (Go) / govulncheck (push) Failing after 1m0s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 26m46s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Failing after 51s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Failing after 1m51s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 26m10s
DB-Integrationstests / db-tests (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (push) Canceled after 0s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Successful in 25m41s
1de1da433f
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
patrick merged commit 9bb138166d into main 2026-10-28 23:30:24 +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#419