RBAC #369: Rolle aus dem Token, Policy-Pruefung fail-closed #420

Merged
patrick merged 1 commits from fix/rbac-role-failclosed into main 2026-10-28 23:29:22 +01:00
Owner

Baut auf #418 (Basis-Branch fix/issue-369-rbac). Nach dem Merge von #418 den PR auf main umhaengen; der Diff enthaelt dann nur diesen einen Commit.

Aenderung

  • workflow/internal/bearerauth: setzt X-User-Role aus den Token-Claims. Ein vom Client gesendeter X-User-Role wird immer entfernt (auch im Uebergangsmodus).
  • taskapi, connectorsetupapi: Policy-Abfrage mit dieser Rolle, ohne Rolle user.
  • Migration 0029 (noch nicht gemergt, daher direkt angepasst): connect.setup.write nur noch fuer tenant_admin. workflow.task.write bleibt bei user, weil Bearbeiter ihre Aufgaben selbst setzen.
  • fail-closed: ai-app, workflow-app, connect-app starten ohne NEXARCH_<MODUL>_APP_POLICY_URL nicht mehr, ausser NEXARCH_<MODUL>_APP_POLICY_DISABLED=1 (Warnung im Log).
  • Doku: docs/BETRIEB-PORTAL.md, workflow/docs/ENG-11-PRUEFPROTOKOLL.md.

Auswirkung

  • Breaking beim Deploy: ohne Policy-URL (oder …_POLICY_DISABLED=1) startet keiner der drei Dienste.
  • connect-app: die Rolle kommt erst mit #419 (connect/internal/bearerauth, dort ist die gleiche Aenderung als eigener Commit). Bis dahin gilt dort user, also darf niemand connect.setup.write.
  • ai-app wird per Service-Token von Modulen gerufen, ohne Nutzerrolle; es fragt weiter mit user.
  • statusapi fragt weiter mit fester Rolle user (nicht Teil dieses PR).

Tests (131)

workflow: bearerauth (Rolle nur aus Token, Client-Rolle entfernt), taskapi, pflichttestgate; connect: connectorsetupapi, statusapi; ai: cmd/ai-app (Policy erforderlich ausser DISABLED), semanticsearchapi; go vet und gofmt sauber. Nicht gelaufen: Tests mit echter Policy-API und DB-Tests der Migration (Aenderung ist ein entfernter VALUES-Eintrag).

🤖 Generated with Claude Code

**Baut auf #418** (Basis-Branch `fix/issue-369-rbac`). Nach dem Merge von #418 den PR auf `main` umhaengen; der Diff enthaelt dann nur diesen einen Commit. ## Aenderung - `workflow/internal/bearerauth`: setzt `X-User-Role` aus den Token-Claims. Ein vom Client gesendeter `X-User-Role` wird immer entfernt (auch im Uebergangsmodus). - `taskapi`, `connectorsetupapi`: Policy-Abfrage mit dieser Rolle, ohne Rolle `user`. - Migration 0029 (noch nicht gemergt, daher direkt angepasst): `connect.setup.write` nur noch fuer `tenant_admin`. `workflow.task.write` bleibt bei `user`, weil Bearbeiter ihre Aufgaben selbst setzen. - fail-closed: `ai-app`, `workflow-app`, `connect-app` starten ohne `NEXARCH_<MODUL>_APP_POLICY_URL` nicht mehr, ausser `NEXARCH_<MODUL>_APP_POLICY_DISABLED=1` (Warnung im Log). - Doku: `docs/BETRIEB-PORTAL.md`, `workflow/docs/ENG-11-PRUEFPROTOKOLL.md`. ## Auswirkung - **Breaking beim Deploy:** ohne Policy-URL (oder `…_POLICY_DISABLED=1`) startet keiner der drei Dienste. - `connect-app`: die Rolle kommt erst mit #419 (`connect/internal/bearerauth`, dort ist die gleiche Aenderung als eigener Commit). Bis dahin gilt dort `user`, also darf niemand `connect.setup.write`. - `ai-app` wird per Service-Token von Modulen gerufen, ohne Nutzerrolle; es fragt weiter mit `user`. - `statusapi` fragt weiter mit fester Rolle `user` (nicht Teil dieses PR). ## Tests (131) workflow: `bearerauth` (Rolle nur aus Token, Client-Rolle entfernt), `taskapi`, `pflichttestgate`; connect: `connectorsetupapi`, `statusapi`; ai: `cmd/ai-app` (Policy erforderlich ausser DISABLED), `semanticsearchapi`; `go vet` und `gofmt` sauber. Nicht gelaufen: Tests mit echter Policy-API und DB-Tests der Migration (Aenderung ist ein entfernter VALUES-Eintrag). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
patrick changed target branch from fix/issue-369-rbac to main 2026-10-28 23:29:12 +01:00
patrick added 1 commit 2026-10-28 23:29:12 +01:00
RBAC #369: Rolle aus dem Token durchreichen, Policy-Pruefung fail-closed
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 1m6s
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 1m23s
CI / Web (pdf-viewer) (push) Failing after 1m2s
CI / Web (shl) (push) Failing after 16s
Supply-Chain-Scan (Go) / govulncheck (push) Failing after 1m2s
AI-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 24m42s
CI / Core-Modul (Build/Vet/Test) (pull_request) Successful in 1m0s
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 5s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Failing after 25s
CI / Web (dms-browser) (pull_request) Failing after 1m32s
CI / Web (pdf-viewer) (pull_request) Failing after 55s
CI / Web (shl) (pull_request) Failing after 13s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 24m26s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Failing after 1m2s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Failing after 1m52s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 24m43s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Successful in 25m10s
Workflow-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 24m45s
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
698f0490f2
workflow bearerauth setzt X-User-Role nur aus den Claims (Client-Header werden
entfernt); taskapi und connectorsetupapi fragen mit dieser Rolle, Default user.
Migration 0029: connect.setup.write nur fuer tenant_admin.
ai-app, workflow-app, connect-app starten ohne POLICY_URL nicht mehr, ausser
mit NEXARCH_<MODUL>_APP_POLICY_DISABLED=1.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
patrick merged commit 1b095507f3 into main 2026-10-28 23:29:22 +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#420