workflow: Bearer-Token statt freier X-User-Id (B-06) #408

Merged
patrick merged 1 commits from feat/workflow-bearer into main 2026-10-28 12:16:36 +01:00
Owner

Karte B-06, Entscheidungen des Users: Verifier aus common, Uebergang mit Schalter.

Aenderung

  • Neues Paket workflow/internal/bearerauth: Middleware vor dem gesamten Mux von workflow-app. Mit Authorization: Bearer wird das Modul-Token ueber common/moduleauth geprueft (Signatur per JWKS, Ablauf, Zielmodul, Aussteller); X-User-Id und X-Tenant-Slug werden aus den Claims gesetzt und ueberschreiben Client-Werte. Ungueltiges Token: 401, Core-Ausfall ohne bekannten Schluesselsatz: 503. Die API-Handler bleiben unveraendert.
  • Ohne Bearer: X-User-Id/X-Tenant-Slug werden entfernt (Handler antworten 401), ausser NEXARCH_WORKFLOW_APP_ALLOW_HEADER_AUTH=1 (Uebergang, loggt eine Warnung).
  • workflow-app startet nur mit NEXARCH_WORKFLOW_APP_JWKS_URL oder dem Uebergangsschalter. Weitere Variablen: ..._JWT_AUDIENCE (Standard workflow), ..._JWT_ISSUER (Standard nexarch-core), ..._JWT_AUDIENCE_ENFORCE=false fuer das Ausrollfenster.
  • common v0.2.1 (moduleauth).

Auswirkungen (Pflicht-Review)

  • Betrieb: Bestehende Deployments starten ohne neue Variable nicht mehr. Sofort-Weg: NEXARCH_WORKFLOW_APP_ALLOW_HEADER_AUTH=1 setzen (altes Verhalten). Zielzustand: JWKS-URL des Core setzen, Core muss workflow in NEXARCH_ACCOUNT_MODULE_AUDIENCES fuehren, Clients holen das Token mit ?module=workflow.
  • Clients: Die Frontend-Proxy-Routen (web/workflow-tasks, workflow-cases, routing-rules, workflow-approvals) und das Portal-BFF senden noch X-User-Id; sie laufen mit dem Schalter weiter und muessen als Folge auf Bearer umgestellt werden. Danach Schalter entfernen.
  • taskapi ist nicht in workflow-app gemountet und von dieser Aenderung nicht erfasst.
  • Nicht aufgenommen: Muster fuer bearerauth im Pflichttest-Gate von workflow (Folgekarte).

Tests

131: build, vet, gofmt sauber; bearerauth (Header-Ueberschreiben, ungueltiges Token, Header-Strip mit/ohne Schalter), cmd/workflow-app und contracttest ok.

🤖 Generated with Claude Code

Karte B-06, Entscheidungen des Users: Verifier aus common, Uebergang mit Schalter. ## Aenderung - Neues Paket `workflow/internal/bearerauth`: Middleware vor dem gesamten Mux von `workflow-app`. Mit `Authorization: Bearer` wird das Modul-Token ueber `common/moduleauth` geprueft (Signatur per JWKS, Ablauf, Zielmodul, Aussteller); `X-User-Id` und `X-Tenant-Slug` werden aus den Claims gesetzt und ueberschreiben Client-Werte. Ungueltiges Token: 401, Core-Ausfall ohne bekannten Schluesselsatz: 503. Die API-Handler bleiben unveraendert. - Ohne Bearer: `X-User-Id`/`X-Tenant-Slug` werden entfernt (Handler antworten 401), ausser `NEXARCH_WORKFLOW_APP_ALLOW_HEADER_AUTH=1` (Uebergang, loggt eine Warnung). - `workflow-app` startet nur mit `NEXARCH_WORKFLOW_APP_JWKS_URL` oder dem Uebergangsschalter. Weitere Variablen: `..._JWT_AUDIENCE` (Standard `workflow`), `..._JWT_ISSUER` (Standard `nexarch-core`), `..._JWT_AUDIENCE_ENFORCE=false` fuer das Ausrollfenster. - `common` v0.2.1 (moduleauth). ## Auswirkungen (Pflicht-Review) - **Betrieb:** Bestehende Deployments starten ohne neue Variable nicht mehr. Sofort-Weg: `NEXARCH_WORKFLOW_APP_ALLOW_HEADER_AUTH=1` setzen (altes Verhalten). Zielzustand: JWKS-URL des Core setzen, Core muss `workflow` in `NEXARCH_ACCOUNT_MODULE_AUDIENCES` fuehren, Clients holen das Token mit `?module=workflow`. - **Clients:** Die Frontend-Proxy-Routen (web/workflow-tasks, workflow-cases, routing-rules, workflow-approvals) und das Portal-BFF senden noch `X-User-Id`; sie laufen mit dem Schalter weiter und muessen als Folge auf Bearer umgestellt werden. Danach Schalter entfernen. - `taskapi` ist nicht in `workflow-app` gemountet und von dieser Aenderung nicht erfasst. - Nicht aufgenommen: Muster fuer `bearerauth` im Pflichttest-Gate von workflow (Folgekarte). ## Tests 131: build, vet, gofmt sauber; `bearerauth` (Header-Ueberschreiben, ungueltiges Token, Header-Strip mit/ohne Schalter), `cmd/workflow-app` und `contracttest` ok. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
patrick added 1 commit 2026-10-28 11:39:13 +01:00
workflow: Bearer-Token-Pruefung vor den APIs (Uebergangsschalter fuer X-User-Id)
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (push) Canceled after 0s
CI / Core-Modul (Build/Vet/Test) (push) Successful in 59s
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 21s
CI / Web (dms-browser) (push) Failing after 1m25s
CI / Web (pdf-viewer) (push) Failing after 1m6s
CI / Web (shl) (push) Failing after 11s
Supply-Chain-Scan (Go) / govulncheck (push) Failing after 1m19s
CI / Core-Modul (Build/Vet/Test) (pull_request) Successful in 1m6s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (pull_request) Successful in 5s
CI / Connect Security-Test-Gate (QA-01 AC4, Merge-Gate) (pull_request) Successful in 5s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Failing after 20s
CI / Web (dms-browser) (pull_request) Failing after 1m27s
CI / Web (pdf-viewer) (pull_request) Failing after 56s
CI / Web (shl) (pull_request) Failing after 14s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 16m39s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Failing after 1m2s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Failing after 1m45s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 17m11s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Successful in 17m44s
Workflow-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 17m29s
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (pull_request) Failing after 33s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (push) Failing after 2s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (pull_request) Failing after 2s
0d07b79b04
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
patrick merged commit e5b37b6278 into main 2026-10-28 12:16:36 +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#408