Files
nexarch/workflow/internal/taskapi/taskapi.go
T
patrickandClaude Sonnet 5.5 0a4c0aaf92
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 38s
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 14s
CI / Web (dms-browser) (push) Failing after 53s
CI / Web (pdf-viewer) (push) Failing after 38s
CI / Web (shl) (push) Failing after 8s
Supply-Chain-Scan (Go) / govulncheck (push) Failing after 37s
AI-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 15m54s
CI / Core-Modul (Build/Vet/Test) (pull_request) Successful in 37s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (pull_request) Successful in 2s
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 13s
CI / Web (dms-browser) (pull_request) Failing after 53s
CI / Web (pdf-viewer) (pull_request) Failing after 38s
CI / Web (shl) (pull_request) Failing after 8s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Successful in 16m2s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Failing after 56s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Failing after 1m13s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Successful in 16m18s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Successful in 15m49s
Workflow-Pflichttest-Gate / pflichttest-gate (pull_request) Failing after 22m6s
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
SEC: RBAC-Pruefung in ai-search, workflow-tasks, connect-setup (#369)
- semanticsearchapi: ai.search.read per policyclient (WithPolicy), Tenant aus
  dem Anfragekoerper, nach Service-Token-Pruefung
- taskapi: workflow.task.read (GET /tasks) und workflow.task.write (PATCH
  status); Paket jetzt in cmd/workflow-app verdrahtet (lag bisher nur im
  a11y-Fixture) und damit hinter bearerauth
- connectorsetupapi: connect.setup.read/write je Route
- Migration 0029 (up/down): neue Rechte fuer user und tenant_admin
- Wiring: NEXARCH_{AI,WORKFLOW,CONNECT}_APP_POLICY_URL/_POLICY_SERVICE_TOKEN;
  ohne URL Warnung und keine Pruefung (Uebergang), URL ohne Token Startfehler;
  connect-app setzt die bestehende statusapi-Pruefung damit ebenfalls scharf
- Tests je API: 403 bei Ablehnung, 200 bei Freigabe, 503 bei Ausfall, Policy
  wird ohne Identitaet/Service-Token nicht befragt
- MODULE_REGISTRY: ai_search_chat, workflow_tasks, connect_setup (+ Icons,
  Portal-Rechteliste)
- Doku: BETRIEB-PORTAL.md, ENG-11 Umgebungsvariablen

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
2026-10-28 22:36:43 +01:00

123 lines
4.0 KiB
Go

// Package taskapi ist die HTTP-Bindung fuer die Aufgaben-Oberflaeche
// (TSK-04): GET /tasks fuer Liste und Kanban-Ansicht zusammen, PATCH
// /tasks/{id}/status fuer die Drag&Drop-Statusaenderung. Nutzeridentitaet
// aus X-User-Id, Tenant-Kontext aus X-Tenant-Slug -- gleiches Muster wie
// backchannelapi (VOR-04/VOR-06): der Tenant-Kontext des anfragenden Nutzers
// muss zur eigenen Instanz passen und optional gegen Core bestaetigt werden,
// bevor Aufgaben zurueckgegeben werden (Akzeptanzkriterium 5). In
// cmd/workflow-app setzt bearerauth beide Header aus dem verifizierten
// Token. Issue #369: zusaetzlich Rechtepruefung workflow.task.read/write.
package taskapi
import (
"encoding/json"
"errors"
"log"
"net/http"
"gitea.perlbach24.de/scripte/nexarch/common/policyclient"
"gitea.perlbach24.de/scripte/nexarch/workflow/internal/task"
)
const (
permissionTaskRead = "workflow.task.read"
permissionTaskWrite = "workflow.task.write"
)
type Server struct {
store *task.Store
ownTenantSlug string
policy *policyclient.Client
}
func NewServer(store *task.Store, ownTenantSlug string) *Server {
return &Server{store: store, ownTenantSlug: ownTenantSlug}
}
// WithPolicy aktiviert die Rechtepruefung gegen Core (POST /authorize).
// Rolle ist fest "user": bearerauth reicht nur Subject und Tenant aus den
// Claims durch, keine Rolle (Uebergang, wie connect/internal/statusapi).
func (s *Server) WithPolicy(client *policyclient.Client) *Server {
s.policy = client
return s
}
func (s *Server) RegisterRoutes(mux *http.ServeMux) {
mux.HandleFunc("GET /tasks", s.requireAuth(permissionTaskRead, s.handleList))
mux.HandleFunc("PATCH /tasks/{id}/status", s.requireAuth(permissionTaskWrite, s.handleSetStatus))
}
func (s *Server) requireAuth(permission string, next http.HandlerFunc) http.HandlerFunc {
return func(w http.ResponseWriter, r *http.Request) {
if r.Header.Get("X-User-Id") == "" {
writeError(w, http.StatusUnauthorized, "X-User-Id fehlt")
return
}
tenantSlug := r.Header.Get("X-Tenant-Slug")
if tenantSlug == "" {
writeError(w, http.StatusUnauthorized, "X-Tenant-Slug fehlt")
return
}
if tenantSlug != s.ownTenantSlug {
log.Printf("taskapi: tenant-kontext abgelehnt (angefragt=%q, instanz=%q)", tenantSlug, s.ownTenantSlug)
writeError(w, http.StatusForbidden, "nicht berechtigt")
return
}
if s.policy != nil {
allowed, err := s.policy.Allowed(r.Context(), tenantSlug, "user", permission)
if err != nil {
writeError(w, http.StatusServiceUnavailable, "berechtigungsdienst nicht erreichbar")
return
}
if !allowed {
writeError(w, http.StatusForbidden, "keine berechtigung")
return
}
}
next(w, r)
}
}
func (s *Server) handleList(w http.ResponseWriter, r *http.Request) {
tasks, err := s.store.ListAll(r.Context())
if err != nil {
writeError(w, http.StatusInternalServerError, "aufgaben konnten nicht gelesen werden")
return
}
writeJSON(w, http.StatusOK, tasks)
}
type setStatusRequest struct {
Status string `json:"status"`
}
func (s *Server) handleSetStatus(w http.ResponseWriter, r *http.Request) {
id := r.PathValue("id")
var req setStatusRequest
if err := json.NewDecoder(r.Body).Decode(&req); err != nil {
writeError(w, http.StatusBadRequest, "ungueltige anfrage")
return
}
err := s.store.SetStatus(r.Context(), id, task.Status(req.Status))
switch {
case err == nil:
w.WriteHeader(http.StatusNoContent)
case errors.Is(err, task.ErrInvalidStatus):
writeError(w, http.StatusBadRequest, "ungueltiger status")
case errors.Is(err, task.ErrNotFound):
writeError(w, http.StatusNotFound, "aufgabe nicht gefunden")
default:
writeError(w, http.StatusInternalServerError, "status konnte nicht gesetzt werden")
}
}
func writeJSON(w http.ResponseWriter, status int, v any) {
w.Header().Set("Content-Type", "application/json")
w.WriteHeader(status)
_ = json.NewEncoder(w).Encode(v)
}
func writeError(w http.ResponseWriter, status int, msg string) {
writeJSON(w, status, map[string]string{"error": msg})
}