dms/backup-tool: Quelle und Ziel waehlbar lokal oder S3

Backup und Restore nutzen NewDriverFromEnv mit den Praefixen
NEXARCH_DMS_BACKUP_ und NEXARCH_DMS_RESTORE_. Ohne STORAGE_DRIVER bleibt
es beim lokalen Verzeichnis. Der Drill-Test laeuft mit TEST_S3_SRC_BUCKET
und TEST_S3_BUCKET gegen S3.

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

common/cmd/pflichttestgate/main.go

@@ -20,6 +20,7 @@ var sensitive = []*regexp.Regexp{
 	regexp.MustCompile(`(^|/)common/policyclient/`),
 	regexp.MustCompile(`(^|/)common/apierror/`),
 	regexp.MustCompile(`(^|/)common/httpharden/`),
+	regexp.MustCompile(`(^|/)common/bearerauth/`),
 }
 
 func main() {

connect/cmd/connect-app/main.go

@@ -15,12 +15,15 @@ import (
 
 	"github.com/jackc/pgx/v5/pgxpool"
 
+	"gitea.perlbach24.de/scripte/nexarch/common/bearerauth"
 	"gitea.perlbach24.de/scripte/nexarch/common/moduleauth"
 	"gitea.perlbach24.de/scripte/nexarch/common/policyclient"
-	"gitea.perlbach24.de/scripte/nexarch/connect/internal/bearerauth"
+	"gitea.perlbach24.de/scripte/nexarch/connect/internal/archiveclient"
 	"gitea.perlbach24.de/scripte/nexarch/connect/internal/connectorsetupapi"
 	"gitea.perlbach24.de/scripte/nexarch/connect/internal/credential"
 	"gitea.perlbach24.de/scripte/nexarch/connect/internal/crypto"
+	"gitea.perlbach24.de/scripte/nexarch/connect/internal/datevapi"
+	"gitea.perlbach24.de/scripte/nexarch/connect/internal/datevexport"
 	"gitea.perlbach24.de/scripte/nexarch/connect/internal/dms2import"
 	"gitea.perlbach24.de/scripte/nexarch/connect/internal/dms2importapi"
 	"gitea.perlbach24.de/scripte/nexarch/connect/internal/fieldmapping"
@@ -56,6 +59,25 @@ func buildPolicyClient() *policyclient.Client {
 	return policyclient.New(policyURL, token)
 }
 
+// buildDatevExports liefert nil ohne NEXARCH_CONNECT_APP_ARCHIVE_URL; dann ist
+// /datev/exports nicht registriert. Die DATEV-Uebertragung ist hier nicht verdrahtet.
+func buildDatevExports(pool *pgxpool.Pool) *datevexport.Store {
+	archiveURL := os.Getenv("NEXARCH_CONNECT_APP_ARCHIVE_URL")
+	if archiveURL == "" {
+		log.Print("connect-app: NEXARCH_CONNECT_APP_ARCHIVE_URL fehlt, /datev/exports nicht registriert")
+		return nil
+	}
+	token := os.Getenv("NEXARCH_CONNECT_APP_ARCHIVE_SERVICE_TOKEN")
+	if token == "" {
+		log.Fatal("connect-app: NEXARCH_CONNECT_APP_ARCHIVE_SERVICE_TOKEN ist erforderlich, wenn NEXARCH_CONNECT_APP_ARCHIVE_URL gesetzt ist")
+	}
+	return &datevexport.Store{
+		Pool:        pool,
+		Archive:     archiveclient.New(archiveURL, token),
+		CallbackURL: os.Getenv("NEXARCH_CONNECT_APP_DATEV_CALLBACK_URL"),
+	}
+}
+
 func main() {
 	if !killswitch.Enabled() {
 		log.Printf("connect-app: deaktiviert (%s=false), kein Start", killswitch.EnvEnabled)
@@ -98,6 +120,9 @@ func main() {
 	statusapi.NewServer(syncjob.NewStore(pool)).WithPolicy(policy).RegisterRoutes(mux)
 	dms2importapi.NewServer(mappings, dms2import.NewStore(pool)).RegisterRoutes(mux)
 	connectorsetupapi.NewServer(credentials, mappings).WithPolicy(policy).RegisterRoutes(mux)
+	if exports := buildDatevExports(pool); exports != nil {
+		datevapi.NewServer(exports, nil).WithPolicy(policy).RegisterRoutes(mux)
+	}
 
 	srv := &http.Server{
 		Addr:              addr,

connect/go.mod

@@ -5,7 +5,7 @@ go 1.25.0
 toolchain go1.25.13
 
 require (
-	gitea.perlbach24.de/scripte/nexarch/common v0.2.1
+	gitea.perlbach24.de/scripte/nexarch/common v0.2.2
 	github.com/jackc/pgx/v5 v5.9.2
 	golang.org/x/text v0.29.0
 )

connect/go.sum

@@ -1,5 +1,5 @@
-gitea.perlbach24.de/scripte/nexarch/common v0.2.1 h1:vIreSocK/F4+uid+lZ7u0CSNlPFvA1N7A77k+RhNeAc=
-gitea.perlbach24.de/scripte/nexarch/common v0.2.1/go.mod h1:CiEImG++QMt1xZqSMeDJhPiB7sfWTLZOwjS6OY/Iguc=
+gitea.perlbach24.de/scripte/nexarch/common v0.2.2 h1:PupmTMm3xysOP8J+zgqRAgwro1XlaQn0njsuUgbqN14=
+gitea.perlbach24.de/scripte/nexarch/common v0.2.2/go.mod h1:CiEImG++QMt1xZqSMeDJhPiB7sfWTLZOwjS6OY/Iguc=
 github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=

connect/internal/bearerauth/bearerauth.go

@@ -1,71 +0,0 @@
-// Package bearerauth setzt die Identitaet der Connect-APIs aus einem
-// verifizierten Core-Modul-Token. Die Handler lesen weiter X-User-Id und
-// X-Tenant-Slug; die Middleware ueberschreibt beide aus den Claims, sodass ein
-// Client sie nicht frei setzen kann. X-User-Role kommt nur aus dem Token.
-package bearerauth
-
-import (
-	"encoding/json"
-	"errors"
-	"net/http"
-	"strings"
-
-	"gitea.perlbach24.de/scripte/nexarch/common/moduleauth"
-)
-
-const (
-	headerUser   = "X-User-Id"
-	headerTenant = "X-Tenant-Slug"
-	headerRole   = "X-User-Role"
-)
-
-// Middleware verlangt fuer Anfragen mit Authorization: Bearer ein gueltiges
-// Token. Ohne Bearer werden X-User-Id und X-Tenant-Slug entfernt, ausser
-// allowHeaderAuth ist gesetzt (Uebergang, bis alle Clients Bearer senden).
-func Middleware(v *moduleauth.Verifier, allowHeaderAuth bool, next http.Handler) http.Handler {
-	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		r.Header.Del(headerRole)
-		token, ok := bearer(r)
-		if !ok {
-			if !allowHeaderAuth {
-				r.Header.Del(headerUser)
-				r.Header.Del(headerTenant)
-			}
-			next.ServeHTTP(w, r)
-			return
-		}
-		if v == nil {
-			writeError(w, http.StatusUnauthorized, "nicht angemeldet")
-			return
-		}
-		claims, err := v.Verify(r.Context(), token)
-		if err != nil {
-			if errors.Is(err, moduleauth.ErrInvalidToken) {
-				writeError(w, http.StatusUnauthorized, "nicht angemeldet")
-			} else {
-				writeError(w, http.StatusServiceUnavailable, "identitaetsdienst nicht erreichbar")
-			}
-			return
-		}
-		r.Header.Set(headerUser, claims.Subject)
-		r.Header.Set(headerTenant, claims.Tenant)
-		if claims.Role != "" {
-			r.Header.Set(headerRole, claims.Role)
-		}
-		next.ServeHTTP(w, r)
-	})
-}
-
-func bearer(r *http.Request) (string, bool) {
-	h := r.Header.Get("Authorization")
-	if len(h) > 7 && strings.EqualFold(h[:7], "bearer ") {
-		return strings.TrimSpace(h[7:]), true
-	}
-	return "", false
-}
-
-func writeError(w http.ResponseWriter, status int, message string) {
-	w.Header().Set("Content-Type", "application/json")
-	w.WriteHeader(status)
-	_ = json.NewEncoder(w).Encode(map[string]string{"error": message})
-}

connect/internal/bearerauth/bearerauth_test.go

@@ -1,75 +0,0 @@
-package bearerauth
-
-import (
-	"net/http"
-	"net/http/httptest"
-	"testing"
-
-	"gitea.perlbach24.de/scripte/nexarch/common/moduleauth"
-)
-
-func echo() http.Handler {
-	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		_, _ = w.Write([]byte(r.Header.Get("X-User-Id") + "|" + r.Header.Get("X-Tenant-Slug") + "|" + r.Header.Get("X-User-Role")))
-	})
-}
-
-func do(h http.Handler, hdr map[string]string) *httptest.ResponseRecorder {
-	req := httptest.NewRequest(http.MethodGet, "/x", nil)
-	for k, v := range hdr {
-		req.Header.Set(k, v)
-	}
-	rec := httptest.NewRecorder()
-	h.ServeHTTP(rec, req)
-	return rec
-}
-
-func TestBearerOverridesClientSuppliedHeaders(t *testing.T) {
-	v, iss := moduleauth.NewTestVerifierAndIssuer()
-	h := Middleware(v, false, echo())
-	rec := do(h, map[string]string{
-		"Authorization": "Bearer " + iss.Token("u1", "acme", "user", "", ""),
-		"X-User-Id":     "fremd", "X-Tenant-Slug": "fremd",
-	})
-	if rec.Code != 200 || rec.Body.String() != "u1|acme|user" {
-		t.Fatalf("code=%d body=%q", rec.Code, rec.Body.String())
-	}
-}
-
-func TestInvalidBearerIs401(t *testing.T) {
-	v, _ := moduleauth.NewTestVerifierAndIssuer()
-	_, other := moduleauth.NewTestVerifierAndIssuer()
-	h := Middleware(v, true, echo())
-	if rec := do(h, map[string]string{"Authorization": "Bearer " + other.Token("u1", "acme", "user", "", ""), "X-User-Id": "evil"}); rec.Code != 401 {
-		t.Fatalf("code=%d", rec.Code)
-	}
-	if rec := do(Middleware(nil, true, echo()), map[string]string{"Authorization": "Bearer x.y.z"}); rec.Code != 401 {
-		t.Fatalf("ohne Verifier code=%d", rec.Code)
-	}
-}
-
-func TestWithoutBearerHeadersStrippedUnlessAllowed(t *testing.T) {
-	v, _ := moduleauth.NewTestVerifierAndIssuer()
-	hdr := map[string]string{"X-User-Id": "u9", "X-Tenant-Slug": "t9"}
-	if rec := do(Middleware(v, false, echo()), hdr); rec.Body.String() != "||" {
-		t.Fatalf("Header muessen entfernt werden, bekam %q", rec.Body.String())
-	}
-	if rec := do(Middleware(v, true, echo()), hdr); rec.Body.String() != "u9|t9|" {
-		t.Fatalf("Uebergangsmodus muss Header durchlassen, bekam %q", rec.Body.String())
-	}
-}
-
-func TestRoleOnlyFromToken(t *testing.T) {
-	v, iss := moduleauth.NewTestVerifierAndIssuer()
-	h := Middleware(v, true, echo())
-	rec := do(h, map[string]string{
-		"Authorization": "Bearer " + iss.Token("u1", "acme", "tenant_admin", "", ""),
-		"X-User-Role":   "tenant_admin_fremd",
-	})
-	if rec.Body.String() != "u1|acme|tenant_admin" {
-		t.Fatalf("Rolle muss aus dem Token kommen, bekam %q", rec.Body.String())
-	}
-	if rec := do(h, map[string]string{"X-User-Role": "tenant_admin"}); rec.Body.String() != "||" {
-		t.Fatalf("Rolle ohne Token muss entfernt werden, bekam %q", rec.Body.String())
-	}
-}

connect/internal/datevapi/datevapi.go

@@ -9,6 +9,7 @@ import (
 	"net/http"
 	"time"
 
+	"gitea.perlbach24.de/scripte/nexarch/common/policyclient"
 	"gitea.perlbach24.de/scripte/nexarch/connect/internal/datevexport"
 	"gitea.perlbach24.de/scripte/nexarch/connect/internal/datevtransfer"
 )
@@ -16,6 +17,12 @@ import (
 type Server struct {
 	Exports   *datevexport.Store
 	Transfers *datevtransfer.Store
+	policy    *policyclient.Client
+}
+
+func (s *Server) WithPolicy(client *policyclient.Client) *Server {
+	s.policy = client
+	return s
 }
 
 func NewServer(exports *datevexport.Store, transfers *datevtransfer.Store) *Server {
@@ -23,8 +30,32 @@ func NewServer(exports *datevexport.Store, transfers *datevtransfer.Store) *Serv
 }
 
 func (s *Server) RegisterRoutes(mux *http.ServeMux) {
-	mux.HandleFunc("POST /datev/exports", s.requireIdentity(s.handleTrigger))
-	mux.HandleFunc("GET /datev/exports", s.requireIdentity(s.handleList))
+	mux.HandleFunc("POST /datev/exports", s.requireIdentity(s.requirePermission("connect.datev.write", s.handleTrigger)))
+	mux.HandleFunc("GET /datev/exports", s.requireIdentity(s.requirePermission("connect.datev.read", s.handleList)))
+}
+
+func (s *Server) requirePermission(permission string, next func(w http.ResponseWriter, r *http.Request, tenantSlug string)) func(w http.ResponseWriter, r *http.Request, tenantSlug string) {
+	return func(w http.ResponseWriter, r *http.Request, tenantSlug string) {
+		if s.policy != nil {
+			allowed, err := s.policy.Allowed(r.Context(), tenantSlug, roleOf(r), permission)
+			if err != nil {
+				writeError(w, http.StatusServiceUnavailable, "berechtigungsdienst nicht erreichbar")
+				return
+			}
+			if !allowed {
+				writeError(w, http.StatusForbidden, "keine berechtigung")
+				return
+			}
+		}
+		next(w, r, tenantSlug)
+	}
+}
+
+func roleOf(r *http.Request) string {
+	if role := r.Header.Get("X-User-Role"); role != "" {
+		return role
+	}
+	return "user"
 }
 
 func (s *Server) requireIdentity(next func(w http.ResponseWriter, r *http.Request, tenantSlug string)) http.HandlerFunc {

connect/internal/datevapi/datevapi_policy_test.go

@@ -0,0 +1,46 @@
+package datevapi
+
+import (
+	"encoding/json"
+	"net/http"
+	"net/http/httptest"
+	"strings"
+	"testing"
+
+	"gitea.perlbach24.de/scripte/nexarch/common/policyclient"
+)
+
+func TestPolicy_PermissionAndRoleReachCore(t *testing.T) {
+	type ask struct{ role, permission string }
+	var got []ask
+	policyAPI := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		var body struct {
+			Role       string `json:"role"`
+			Permission string `json:"permission"`
+		}
+		_ = json.NewDecoder(r.Body).Decode(&body)
+		got = append(got, ask{body.Role, body.Permission})
+		_ = json.NewEncoder(w).Encode(map[string]bool{"allowed": false})
+	}))
+	defer policyAPI.Close()
+	mux := http.NewServeMux()
+	(&Server{}).WithPolicy(policyclient.New(policyAPI.URL, "tok")).RegisterRoutes(mux)
+
+	for _, c := range []struct{ method, role string }{{"GET", "tenant_admin"}, {"POST", ""}} {
+		req := httptest.NewRequest(c.method, "/datev/exports", strings.NewReader("{}"))
+		req.Header.Set("X-User-Id", "u1")
+		req.Header.Set("X-Tenant-Slug", "acme")
+		if c.role != "" {
+			req.Header.Set("X-User-Role", c.role)
+		}
+		rec := httptest.NewRecorder()
+		mux.ServeHTTP(rec, req)
+		if rec.Code != http.StatusForbidden {
+			t.Fatalf("%s: erwartet 403, bekam %d", c.method, rec.Code)
+		}
+	}
+	want := []ask{{"tenant_admin", "connect.datev.read"}, {"user", "connect.datev.write"}}
+	if len(got) != 2 || got[0] != want[0] || got[1] != want[1] {
+		t.Fatalf("erwartet %v, bekam %v", want, got)
+	}
+}

connect/internal/statusapi/statusapi.go

@@ -22,8 +22,7 @@ type Server struct {
 
 // WithPolicy aktiviert RBAC-12: echte Rechtepruefung gegen Core (POST
 // /authorize), zusaetzlich zur bestehenden X-User-Id/X-Tenant-Slug-Pruefung.
-// Rolle ist mangels eigenem Rollenkonzept in Connect fest "user" (Uebergang,
-// wie beim restlichen Header-basierten Identitaetsmodell dieses Pakets).
+// Die Rolle kommt aus X-User-Role (setzt bearerauth aus dem Token), sonst "user".
 func (s *Server) WithPolicy(client *policyclient.Client) *Server {
 	s.policy = client
 	return s
@@ -40,7 +39,7 @@ func (s *Server) RegisterRoutes(mux *http.ServeMux) {
 func (s *Server) requirePermission(permission string, next func(w http.ResponseWriter, r *http.Request, tenantSlug string)) func(w http.ResponseWriter, r *http.Request, tenantSlug string) {
 	return func(w http.ResponseWriter, r *http.Request, tenantSlug string) {
 		if s.policy != nil {
-			allowed, err := s.policy.Allowed(r.Context(), tenantSlug, "user", permission)
+			allowed, err := s.policy.Allowed(r.Context(), tenantSlug, roleOf(r), permission)
 			if err != nil {
 				writeError(w, http.StatusServiceUnavailable, "berechtigungsdienst nicht erreichbar")
 				return
@@ -54,6 +53,13 @@ func (s *Server) requirePermission(permission string, next func(w http.ResponseW
 	}
 }
 
+func roleOf(r *http.Request) string {
+	if role := r.Header.Get("X-User-Role"); role != "" {
+		return role
+	}
+	return "user"
+}
+
 func (s *Server) requireIdentity(next func(w http.ResponseWriter, r *http.Request, tenantSlug string)) http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
 		if r.Header.Get("X-User-Id") == "" {

connect/internal/statusapi/statusapi_policy_test.go

@@ -60,3 +60,32 @@ func TestHandleList_PolicyAllows_Passes(t *testing.T) {
 		t.Fatalf("erwartet 200 mit berechtigung, bekam %d: %s", rec.Code, rec.Body.String())
 	}
 }
+
+func TestHandleList_RoleFromHeaderReachesPolicy(t *testing.T) {
+	var got []string
+	policyAPI := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		var body struct {
+			Role string `json:"role"`
+		}
+		_ = json.NewDecoder(r.Body).Decode(&body)
+		got = append(got, body.Role)
+		_ = json.NewEncoder(w).Encode(map[string]bool{"allowed": true})
+	}))
+	defer policyAPI.Close()
+	s := newTestServer(t).WithPolicy(policyclient.New(policyAPI.URL, "test-token"))
+	mux := http.NewServeMux()
+	s.RegisterRoutes(mux)
+
+	for _, role := range []string{"tenant_admin", ""} {
+		req := httptest.NewRequest("GET", "/connectors/status", nil)
+		req.Header.Set("X-User-Id", "u1")
+		req.Header.Set("X-Tenant-Slug", "acme")
+		if role != "" {
+			req.Header.Set("X-User-Role", role)
+		}
+		mux.ServeHTTP(httptest.NewRecorder(), req)
+	}
+	if len(got) != 2 || got[0] != "tenant_admin" || got[1] != "user" {
+		t.Fatalf("erwartet [tenant_admin user], bekam %v", got)
+	}
+}

dms/README.md

@@ -29,6 +29,10 @@ Fachliche Routen (z. B. `POST /retention/destroy-callback`) liegen unter dem Pr
 `scripts/check-route-prefixes.sh`); nur `/healthz` und `/metrics` bleiben auf Wurzelebene. Die bei archive
 registrierte Rückruf-URL (`NEXARCH_DMS_APP_RETENTION_CALLBACK_URL`) muss den Präfix enthalten.
 
+## Objekt-Speicher
+
+`NEXARCH_DMS_APP_STORAGE_DRIVER` und `NEXARCH_DMS_WORKER_STORAGE_DRIVER` wählen den Treiber: `local` (Standard, braucht `…_STORAGE_DIR`) oder `s3`. Für `s3` sind `…_S3_BUCKET`, `…_S3_ACCESS_KEY_ID` und `…_S3_SECRET_ACCESS_KEY` Pflicht; optional `…_S3_REGION` (Standard `us-east-1`), `…_S3_ENDPOINT_URL` (MinIO, Ceph) und `…_S3_PATH_STYLE=1`. App und Worker müssen denselben Bucket nutzen. Das `backup-tool` liest dieselben Variablen mit den Präfixen `NEXARCH_DMS_BACKUP_` (Quelle) und `NEXARCH_DMS_RESTORE_` (Ziel), auch mit `…_STORAGE_DRIVER=s3`.
+
 ## Struktur
 
 - `cmd/app` — Anfrage-Dienst (HTTP), blockiert nie durch lange Aufgaben

dms/cmd/app/api.go

@@ -41,10 +41,10 @@ type uploadStack struct {
 }
 
 func newUploadStack(pool *pgxpool.Pool, stagingDir string) uploadStack {
-	storageDir := os.Getenv("NEXARCH_DMS_APP_STORAGE_DIR")
-	signingSecret := os.Getenv("NEXARCH_DMS_APP_STORAGE_SIGNING_SECRET")
-	publicBaseURL := os.Getenv("NEXARCH_DMS_APP_STORAGE_PUBLIC_BASE_URL")
-	driver := storage.NewLocalDriver(storageDir, []byte(signingSecret), publicBaseURL, signedURLMaxTTL(os.Getenv("NEXARCH_DMS_APP_SIGNED_URL_MAX_TTL")))
+	driver, err := storage.NewDriverFromEnv(context.Background(), "NEXARCH_DMS_APP")
+	if err != nil {
+		log.Fatalf("dms-app: %v", err)
+	}
 	usageReporter := storage.NewHTTPUsageReporter(
 		os.Getenv("NEXARCH_DMS_APP_USAGE_ENDPOINT_URL"),
 		os.Getenv("NEXARCH_DMS_APP_USAGE_CLIENT_ID"),

dms/cmd/app/main.go

@@ -194,10 +194,10 @@ func setupRetention(mux *http.ServeMux, dsn string) {
 		log.Fatalf("dms-app: datenbankverbindung: %v", err)
 	}
 
-	storageDir := os.Getenv("NEXARCH_DMS_APP_STORAGE_DIR")
-	signingSecret := os.Getenv("NEXARCH_DMS_APP_STORAGE_SIGNING_SECRET")
-	publicBaseURL := os.Getenv("NEXARCH_DMS_APP_STORAGE_PUBLIC_BASE_URL")
-	driver := storage.NewLocalDriver(storageDir, []byte(signingSecret), publicBaseURL, signedURLMaxTTL(os.Getenv("NEXARCH_DMS_APP_SIGNED_URL_MAX_TTL")))
+	driver, err := storage.NewDriverFromEnv(ctx, "NEXARCH_DMS_APP")
+	if err != nil {
+		log.Fatalf("dms-app: %v", err)
+	}
 
 	usageEndpoint := os.Getenv("NEXARCH_DMS_APP_USAGE_ENDPOINT_URL")
 	usageClientID := os.Getenv("NEXARCH_DMS_APP_USAGE_CLIENT_ID")
@@ -360,11 +360,3 @@ func setupWatchFolder(ctx context.Context, dir string) {
 		}
 	}()
 }
-
-func signedURLMaxTTL(raw string) time.Duration {
-	ttl, err := storage.ParseMaxSignedURLTTL(raw)
-	if err != nil {
-		log.Fatalf("dms-app: %v", err)
-	}
-	return ttl
-}

dms/cmd/backup-tool/backup.go

@@ -14,6 +14,7 @@ import (
 type backupConfig struct {
 	DSN        string
 	StorageDir string
+	Driver     storage.Driver
 	TenantSlug string
 	OutDir     string
 	PgDumpPath string
@@ -45,7 +46,10 @@ func runBackup(ctx context.Context, cfg backupConfig) error {
 	}
 	defer pool.Close()
 
-	driver := storage.NewLocalDriver(cfg.StorageDir, []byte("backup-tool-unused"), "", 0)
+	driver := cfg.Driver
+	if driver == nil {
+		driver = storage.NewLocalDriver(cfg.StorageDir, []byte("backup-tool-unused"), "", 0)
+	}
 
 	rows, err := pool.Query(ctx, `
 		SELECT r.id, r.storage_key, r.size_bytes

dms/cmd/backup-tool/drill_test.go

@@ -32,6 +32,27 @@ const drillTenantSlug = "fdn27drill"
 // des Tickets): ein echter, unabhaengiger Quell- und Zielbestand auf dem
 // Entwicklungshost, kein simulierter Lauf. Er belegt AK1 bis AK3 in einem
 // zusammenhaengenden Durchlauf und misst RTO/RPO real.
+// drillDriver liefert den lokalen Treiber, oder einen S3-Treiber, wenn die
+// Umgebungsvariable bucketEnv einen Bucket nennt (TEST_S3_ENDPOINT usw. wie
+// in dms/internal/storage).
+func drillDriver(t *testing.T, bucketEnv, dir string) storage.Driver {
+	t.Helper()
+	bucket := os.Getenv(bucketEnv)
+	if bucket == "" {
+		return storage.NewLocalDriver(dir, []byte("secret"), "https://files.example.test", 0)
+	}
+	region := os.Getenv("TEST_S3_REGION")
+	if region == "" {
+		region = "us-east-1"
+	}
+	d, err := storage.NewS3Driver(context.Background(), bucket, region, os.Getenv("TEST_S3_ENDPOINT"),
+		os.Getenv("TEST_S3_ACCESS_KEY_ID"), os.Getenv("TEST_S3_SECRET_ACCESS_KEY"), true, 0)
+	if err != nil {
+		t.Fatalf("s3-treiber: %v", err)
+	}
+	return d
+}
+
 func TestBackupRestoreDrill_RealRoundTrip(t *testing.T) {
 	dsn := os.Getenv("TEST_TENANT_DSN")
 	if dsn == "" {
@@ -84,7 +105,8 @@ func TestBackupRestoreDrill_RealRoundTrip(t *testing.T) {
 	}
 
 	srcStorageDir := t.TempDir()
-	srcStorageSvc := storage.NewService(storage.NewLocalDriver(srcStorageDir, []byte("secret"), "https://files.example.test", 0), drillNoopUsage{}, drillTenantSlug)
+	srcDriver := drillDriver(t, "TEST_S3_SRC_BUCKET", srcStorageDir)
+	srcStorageSvc := storage.NewService(srcDriver, drillNoopUsage{}, drillTenantSlug)
 	uploadSvc := upload.NewService(srcPool, upload.NewSessionStore(srcPool), upload.NewStaging(t.TempDir()), srcStorageSvc, crypt, upload.NewValidator(10*1024*1024, nil), drillTenantSlug)
 
 	const wantDocs = 3
@@ -108,6 +130,7 @@ func TestBackupRestoreDrill_RealRoundTrip(t *testing.T) {
 	if err := runBackup(ctx, backupConfig{
 		DSN:        dsn,
 		StorageDir: srcStorageDir,
+		Driver:     srcDriver,
 		TenantSlug: drillTenantSlug,
 		OutDir:     backupDir,
 		PgSchema:   srcSchema,
@@ -128,11 +151,13 @@ func TestBackupRestoreDrill_RealRoundTrip(t *testing.T) {
 	dstSchema := fmt.Sprintf("fdn27dst_%d", time.Now().UnixNano())
 	t.Cleanup(func() { _, _ = admin.Exec(context.Background(), "DROP SCHEMA IF EXISTS "+dstSchema+" CASCADE") })
 	dstStorageDir := t.TempDir()
+	dstDriver := drillDriver(t, "TEST_S3_BUCKET", dstStorageDir)
 
 	restoreStart := time.Now()
 	if err := runRestore(ctx, restoreConfig{
 		DSN:          dsn,
 		StorageDir:   dstStorageDir,
+		Driver:       dstDriver,
 		InDir:        backupDir,
 		TargetSchema: dstSchema,
 		Crypto:       crypt,
@@ -162,10 +187,9 @@ func TestBackupRestoreDrill_RealRoundTrip(t *testing.T) {
 
 	// -- FDN-23-Integritaetsjob auf dem wiederhergestellten Bestand: muss
 	// "ok" liefern, sonst waere der Restore nur scheinbar vollstaendig --
-	dstStorageSvc := storage.NewLocalDriver(dstStorageDir, []byte("secret"), "https://files.example.test", 0)
 	verifier := &integrity.Verifier{
 		Pool:       dstPool,
-		Storage:    storage.NewService(dstStorageSvc, drillNoopUsage{}, drillTenantSlug),
+		Storage:    storage.NewService(dstDriver, drillNoopUsage{}, drillTenantSlug),
 		Crypto:     crypt,
 		TenantSlug: drillTenantSlug,
 		Config:     integrity.Config{Mode: integrity.ModeFull, Batch: 100},

dms/cmd/backup-tool/main.go

@@ -7,6 +7,7 @@ import (
 	"os"
 
 	dmscrypto "gitea.perlbach24.de/scripte/nexarch/dms/internal/crypto"
+	"gitea.perlbach24.de/scripte/nexarch/dms/internal/storage"
 )
 
 func main() {
@@ -18,9 +19,15 @@ func main() {
 	var err error
 	switch os.Args[1] {
 	case "backup":
-		err = runBackup(ctx, backupConfigFromEnv(os.Args[2]))
+		cfg := backupConfigFromEnv(os.Args[2])
+		if cfg.Driver, err = storage.NewDriverFromEnv(ctx, "NEXARCH_DMS_BACKUP"); err == nil {
+			err = runBackup(ctx, cfg)
+		}
 	case "restore":
-		err = runRestore(ctx, restoreConfigFromEnv(os.Args[2]))
+		cfg := restoreConfigFromEnv(os.Args[2])
+		if cfg.Driver, err = storage.NewDriverFromEnv(ctx, "NEXARCH_DMS_RESTORE"); err == nil {
+			err = runRestore(ctx, cfg)
+		}
 	default:
 		usage()
 		os.Exit(2)

dms/cmd/backup-tool/restore.go

@@ -23,6 +23,7 @@ var errTargetNotEmpty = errors.New("restore: zieldatenbank ist nicht leer (--for
 type restoreConfig struct {
 	DSN           string
 	StorageDir    string
+	Driver        storage.Driver
 	InDir         string
 	PgRestorePath string
 	PsqlPath      string
@@ -69,7 +70,10 @@ func runRestore(ctx context.Context, cfg restoreConfig) error {
 		}
 	}
 
-	driver := storage.NewLocalDriver(cfg.StorageDir, []byte("backup-tool-unused"), "", 0)
+	driver := cfg.Driver
+	if driver == nil {
+		driver = storage.NewLocalDriver(cfg.StorageDir, []byte("backup-tool-unused"), "", 0)
+	}
 	for _, obj := range m.Objects {
 		if err := restoreObject(ctx, driver, cfg.InDir, obj); err != nil {
 			return fmt.Errorf("restore: objekt %q (revision %s): %w", obj.StorageKey, obj.RevisionID, err)

dms/cmd/worker/main.go

@@ -133,14 +133,16 @@ func processRetentionRegisterJobs(ctx context.Context, queue *jobqueue.Queue, cl
 // startJobWorkers verdrahtet alle Fachjobs (FDN-17). Fehlt die Storage-/Schluessel-
 // Konfiguration, werden sie nicht gestartet und das wird gemeldet.
 func startJobWorkers(ctx context.Context, pool *pgxpool.Pool, queue *jobqueue.Queue, tracker *errortracking.Tracker) {
-	storageDir := os.Getenv("NEXARCH_DMS_WORKER_STORAGE_DIR")
 	tenantSlug := os.Getenv("NEXARCH_DMS_WORKER_TENANT_SLUG")
 	kekURL := os.Getenv("NEXARCH_DMS_WORKER_KEK_ENDPOINT_URL")
-	if storageDir == "" || tenantSlug == "" || kekURL == "" {
-		log.Print("dms-worker: NEXARCH_DMS_WORKER_STORAGE_DIR/_TENANT_SLUG/_KEK_ENDPOINT_URL unvollstaendig, Fachjobs (Vorschau, Konvertierung, Virenscan, Export, Texterkennung) werden NICHT verarbeitet")
+	if !storage.ConfiguredFromEnv("NEXARCH_DMS_WORKER") || tenantSlug == "" || kekURL == "" {
+		log.Print("dms-worker: NEXARCH_DMS_WORKER_STORAGE_DIR (oder _STORAGE_DRIVER=s3 mit _S3_BUCKET)/_TENANT_SLUG/_KEK_ENDPOINT_URL unvollstaendig, Fachjobs (Vorschau, Konvertierung, Virenscan, Export, Texterkennung) werden NICHT verarbeitet")
 		return
 	}
-	driver := storage.NewLocalDriver(storageDir, []byte(os.Getenv("NEXARCH_DMS_WORKER_STORAGE_SIGNING_SECRET")), os.Getenv("NEXARCH_DMS_WORKER_STORAGE_PUBLIC_BASE_URL"), signedURLMaxTTL(os.Getenv("NEXARCH_DMS_WORKER_SIGNED_URL_MAX_TTL")))
+	driver, err := storage.NewDriverFromEnv(ctx, "NEXARCH_DMS_WORKER")
+	if err != nil {
+		log.Fatalf("dms-worker: %v", err)
+	}
 	usage := storage.NewHTTPUsageReporter(os.Getenv("NEXARCH_DMS_WORKER_USAGE_ENDPOINT_URL"), os.Getenv("NEXARCH_DMS_WORKER_USAGE_CLIENT_ID"), os.Getenv("NEXARCH_DMS_WORKER_USAGE_CLIENT_SECRET"), nil)
 	cryptoSvc := dmscrypto.NewService(dmscrypto.NewHTTPKEKProvider(kekURL, os.Getenv("NEXARCH_DMS_WORKER_KEK_CLIENT_ID"), os.Getenv("NEXARCH_DMS_WORKER_KEK_CLIENT_SECRET"), nil))
 
@@ -304,11 +306,3 @@ func startJobWorkers(ctx context.Context, pool *pgxpool.Pool, queue *jobqueue.Qu
 		go integrity.RunScheduler(ctx, pool, queue, integritySched, 30*time.Second)
 	}
 }
-
-func signedURLMaxTTL(raw string) time.Duration {
-	ttl, err := storage.ParseMaxSignedURLTTL(raw)
-	if err != nil {
-		log.Fatalf("dms-worker: %v", err)
-	}
-	return ttl
-}

dms/internal/storage/fromenv.go

@@ -0,0 +1,42 @@
+package storage
+
+import (
+	"context"
+	"fmt"
+	"os"
+	"strings"
+)
+
+// NewDriverFromEnv wählt den Treiber über <prefix>_STORAGE_DRIVER ("local",
+// Standard, oder "s3"). Zugangsdaten kommen nur aus der Umgebung.
+func NewDriverFromEnv(ctx context.Context, prefix string) (Driver, error) {
+	env := func(name string) string { return os.Getenv(prefix + "_" + name) }
+	ttl, err := ParseMaxSignedURLTTL(env("SIGNED_URL_MAX_TTL"))
+	if err != nil {
+		return nil, err
+	}
+	switch strings.ToLower(env("STORAGE_DRIVER")) {
+	case "", "local":
+		return NewLocalDriver(env("STORAGE_DIR"), []byte(env("STORAGE_SIGNING_SECRET")), env("STORAGE_PUBLIC_BASE_URL"), ttl), nil
+	case "s3":
+		bucket, key, secret := env("S3_BUCKET"), env("S3_ACCESS_KEY_ID"), env("S3_SECRET_ACCESS_KEY")
+		if bucket == "" || key == "" || secret == "" {
+			return nil, fmt.Errorf("storage: %s_S3_BUCKET, _S3_ACCESS_KEY_ID und _S3_SECRET_ACCESS_KEY sind erforderlich", prefix)
+		}
+		region := env("S3_REGION")
+		if region == "" {
+			region = "us-east-1"
+		}
+		return NewS3Driver(ctx, bucket, region, env("S3_ENDPOINT_URL"), key, secret, env("S3_PATH_STYLE") == "1", ttl)
+	default:
+		return nil, fmt.Errorf("storage: %s_STORAGE_DRIVER %q unbekannt (local oder s3)", prefix, env("STORAGE_DRIVER"))
+	}
+}
+
+// ConfiguredFromEnv meldet, ob für prefix ein Speicher konfiguriert ist.
+func ConfiguredFromEnv(prefix string) bool {
+	if strings.EqualFold(os.Getenv(prefix+"_STORAGE_DRIVER"), "s3") {
+		return os.Getenv(prefix+"_S3_BUCKET") != ""
+	}
+	return os.Getenv(prefix+"_STORAGE_DIR") != ""
+}

dms/internal/storage/fromenv_test.go

@@ -0,0 +1,69 @@
+package storage
+
+import (
+	"context"
+	"testing"
+)
+
+func TestNewDriverFromEnv(t *testing.T) {
+	ctx := context.Background()
+
+	t.Run("Standard ist lokal", func(t *testing.T) {
+		t.Setenv("X_STORAGE_DIR", t.TempDir())
+		d, err := NewDriverFromEnv(ctx, "X")
+		if err != nil {
+			t.Fatal(err)
+		}
+		if _, ok := d.(*LocalDriver); !ok {
+			t.Fatalf("erwartet *LocalDriver, bekam %T", d)
+		}
+	})
+
+	t.Run("s3 mit Zugangsdaten", func(t *testing.T) {
+		t.Setenv("X_STORAGE_DRIVER", "S3")
+		t.Setenv("X_S3_BUCKET", "b")
+		t.Setenv("X_S3_ACCESS_KEY_ID", "k")
+		t.Setenv("X_S3_SECRET_ACCESS_KEY", "s")
+		t.Setenv("X_S3_ENDPOINT_URL", "http://127.0.0.1:9000")
+		d, err := NewDriverFromEnv(ctx, "X")
+		if err != nil {
+			t.Fatal(err)
+		}
+		if _, ok := d.(*S3Driver); !ok {
+			t.Fatalf("erwartet *S3Driver, bekam %T", d)
+		}
+	})
+
+	t.Run("s3 ohne Zugangsdaten schlaegt fehl", func(t *testing.T) {
+		t.Setenv("X_STORAGE_DRIVER", "s3")
+		t.Setenv("X_S3_BUCKET", "b")
+		if _, err := NewDriverFromEnv(ctx, "X"); err == nil {
+			t.Fatal("erwartet Fehler ohne Schluessel")
+		}
+	})
+
+	t.Run("unbekannter Treiber", func(t *testing.T) {
+		t.Setenv("X_STORAGE_DRIVER", "ftp")
+		if _, err := NewDriverFromEnv(ctx, "X"); err == nil {
+			t.Fatal("erwartet Fehler fuer unbekannten Treiber")
+		}
+	})
+}
+
+func TestConfiguredFromEnv(t *testing.T) {
+	if ConfiguredFromEnv("Y") {
+		t.Fatal("ohne Variablen nicht konfiguriert")
+	}
+	t.Setenv("Y_STORAGE_DIR", "/x")
+	if !ConfiguredFromEnv("Y") {
+		t.Fatal("STORAGE_DIR zaehlt als konfiguriert")
+	}
+	t.Setenv("Y_STORAGE_DRIVER", "s3")
+	if ConfiguredFromEnv("Y") {
+		t.Fatal("s3 braucht den Bucket")
+	}
+	t.Setenv("Y_S3_BUCKET", "b")
+	if !ConfiguredFromEnv("Y") {
+		t.Fatal("s3 mit Bucket ist konfiguriert")
+	}
+}

docs/ARBEITSKARTEN.md

@@ -71,9 +71,11 @@ Offene PRs, die auf Merge warten, stehen unter C-01. Merge-Reihenfolge: erst DB-
 
 ## D. Portal-Betrieb (Stand 2026-10-28)
 - D-01 Portal-UI (PR #409) und account-api-Start (PR #410): gemergt.
-- D-02 Portal auf 131: gebaut, Unit und nginx/nft offen, siehe `docs/BETRIEB-PORTAL.md`.
-- D-03 account-api auf 131: DB-Passwort `nexarch_core` abgleichen (Betreiber), Unit aus Vorlage erneuern.
+- D-02 Portal auf 131: eingerichtet mit `deploy/131-portal-einrichten.sh` (#424): Unit `nexarch-web-portal`, Port 8450, CORS am Core, Audience `portal`. Offen: Anmeldung im Browser testen.
+- D-03 account-api auf 131: erledigt, Passwort der Rolle `nexarch_core` abgeglichen (2026-10-29). Offen: `NEXARCH_ACCOUNT_REGISTRY_DSN` fehlt (Sperr-Ereignisse nicht im zentralen Audit-Log).
 - D-04 204: localhost-Adressen in connect/connect-datev/rbac-admin behoben; Port 3113 offen.
 - D-05 Frontends auf Bearer: workflow-approvals, workflow-tasks, workflow-cases, routing-rules, Portal-BFF (PR #412, #413), mail-search (#416), connect-setup, connect-status, datev-export, dms2-import mit `connect-app`-Prüfung (#419); gemeinsamer Helfer `createBearerClient` in `@nexarch/shl` (#417). Offen: Ende-zu-Ende-Test mit Anmeldung auf 204; CORS für `Authorization` an der Connect-API auf 131; `connect` in `NEXARCH_ACCOUNT_MODULE_AUDIENCES`; `NEXARCH_CONNECT_APP_JWKS_URL` und `_TENANT_SLUG` setzen; danach `NEXARCH_WORKFLOW_APP_ALLOW_HEADER_AUTH` und `NEXARCH_CONNECT_APP_ALLOW_HEADER_AUTH` entfernen.
 - D-06 Workflow-Demo auf 204 läuft (siehe `docs/BETRIEB-PORTAL.md`).
 - D-07 Issue #369 (RBAC in ai-search, workflow-tasks, connect-setup): gemergt (#418, #420). Rolle kommt als `X-User-Role` aus dem Token; `connect.setup.write` nur `tenant_admin`. Dienste starten ohne `NEXARCH_{AI,WORKFLOW,CONNECT}_APP_POLICY_URL` nicht (Übergang: `…_POLICY_DISABLED=1`). Offen: Migration 0029 auf allen Mandanten-DBs einspielen, bevor die Binaries ausgerollt werden; `statusapi` fragt weiter mit fester Rolle `user`; `bearerauth` liegt doppelt (workflow, connect) und gehört nach `common/`; `datevapi` ist in `connect-app` nicht registriert.
+- D-08 Login und Portal (#422, #423): nach der Anmeldung direkt ins Portal (`NEXT_PUBLIC_PORTAL_URL`, ohne Rücksprung-URL wegen Open-Redirect); Betriebszustand an den Kacheln, nur für `core.tenant.admin` sichtbar, braucht `PORTAL_STATUS_API_URL`. Offen: Statusdienst-Demo auf 204.
+- D-09 Archive-Jobs auf 131 (2026-10-29): `backup-full` scheiterte mit falschem Passwort der Rolle `nexarch_backup`, `backup-incremental` als Folge; `reconcile` und `scrub` zeigten auf die Test-DB `dms_tenant_test` und liefen auf `file_revisions`-Fehler. Behoben: Passwort abgeglichen, DSN und Storage-Pfad auf die DB von `dms-app` (`tenant_rollout_test`), `scrub_state` angelegt. Offen: eigene Leserolle statt `dms_app`; klären, ob `tenant_rollout_test` die endgültige Mandanten-DB ist; Dauer des Backup-Ausfalls unbekannt (Fehlstarts seit mindestens 22.10.).

workflow/cmd/workflow-app/main.go

@@ -16,6 +16,7 @@ import (
 
 	"github.com/jackc/pgx/v5/pgxpool"
 
+	"gitea.perlbach24.de/scripte/nexarch/common/bearerauth"
 	"gitea.perlbach24.de/scripte/nexarch/common/moduleauth"
 	"gitea.perlbach24.de/scripte/nexarch/common/policyclient"
 	"gitea.perlbach24.de/scripte/nexarch/common/tenantregistryclient"
@@ -23,7 +24,6 @@ import (
 	"gitea.perlbach24.de/scripte/nexarch/workflow/internal/approvalapi"
 	"gitea.perlbach24.de/scripte/nexarch/workflow/internal/backchannel"
 	"gitea.perlbach24.de/scripte/nexarch/workflow/internal/backchannelapi"
-	"gitea.perlbach24.de/scripte/nexarch/workflow/internal/bearerauth"
 	"gitea.perlbach24.de/scripte/nexarch/workflow/internal/caseapi"
 	"gitea.perlbach24.de/scripte/nexarch/workflow/internal/caseobj"
 	"gitea.perlbach24.de/scripte/nexarch/workflow/internal/caseoverview"

workflow/docs/TESTSTRATEGIE.md

@@ -64,8 +64,9 @@ aendert — **jede geaenderte `.go`-Datei muss von einer geaenderten oder neuen
 - **Auth-nahe Logik:** `workflow/internal/svcauth/` (Service-Token-Vergleich in
   konstanter Zeit), `workflow/internal/moduleadapter/` (prueft `X-Service-Token`
   gegen `svcauth.Equal` — der tatsaechliche Durchsetzungspunkt),
-  `workflow/internal/bearerauth/` (prueft das Bearer-Token der Frontends gegen
-  die Core-JWKS; `X-User-Id` nur noch ueber den Uebergangsschalter)
+  `common/bearerauth/` (prueft das Bearer-Token der Frontends gegen die
+  Core-JWKS; `X-User-Id` nur noch ueber den Uebergangsschalter; Gate im
+  common-Modul)
 - **Tenant-Scoping:** `workflow/internal/tenantregistryclient/` (fragt Core TEN-01,
   muss bei Fehler fail-closed bleiben), `workflow/internal/backchannel/` und
   `workflow/internal/backchannelapi/` (extern erreichbarer Eingang, der den

workflow/go.mod

@@ -3,7 +3,7 @@ module gitea.perlbach24.de/scripte/nexarch/workflow
 go 1.25.0
 
 require (
-	gitea.perlbach24.de/scripte/nexarch/common v0.2.1
+	gitea.perlbach24.de/scripte/nexarch/common v0.2.2
 	github.com/jackc/pgx/v5 v5.9.2
 )
 

workflow/go.sum

@@ -1,5 +1,5 @@
-gitea.perlbach24.de/scripte/nexarch/common v0.2.1 h1:vIreSocK/F4+uid+lZ7u0CSNlPFvA1N7A77k+RhNeAc=
-gitea.perlbach24.de/scripte/nexarch/common v0.2.1/go.mod h1:CiEImG++QMt1xZqSMeDJhPiB7sfWTLZOwjS6OY/Iguc=
+gitea.perlbach24.de/scripte/nexarch/common v0.2.2 h1:PupmTMm3xysOP8J+zgqRAgwro1XlaQn0njsuUgbqN14=
+gitea.perlbach24.de/scripte/nexarch/common v0.2.2/go.mod h1:CiEImG++QMt1xZqSMeDJhPiB7sfWTLZOwjS6OY/Iguc=
 github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=

workflow/internal/bearerauth/bearerauth.go

@@ -1,71 +0,0 @@
-// Package bearerauth setzt die Identitaet der Workflow-APIs aus einem
-// verifizierten Core-Modul-Token. Die Handler lesen weiter X-User-Id und
-// X-Tenant-Slug; die Middleware ueberschreibt beide aus den Claims, sodass ein
-// Client sie nicht frei setzen kann. X-User-Role kommt nur aus dem Token.
-package bearerauth
-
-import (
-	"encoding/json"
-	"errors"
-	"net/http"
-	"strings"
-
-	"gitea.perlbach24.de/scripte/nexarch/common/moduleauth"
-)
-
-const (
-	headerUser   = "X-User-Id"
-	headerTenant = "X-Tenant-Slug"
-	headerRole   = "X-User-Role"
-)
-
-// Middleware verlangt fuer Anfragen mit Authorization: Bearer ein gueltiges
-// Token. Ohne Bearer werden X-User-Id und X-Tenant-Slug entfernt, ausser
-// allowHeaderAuth ist gesetzt (Uebergang, bis alle Clients Bearer senden).
-func Middleware(v *moduleauth.Verifier, allowHeaderAuth bool, next http.Handler) http.Handler {
-	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		r.Header.Del(headerRole)
-		token, ok := bearer(r)
-		if !ok {
-			if !allowHeaderAuth {
-				r.Header.Del(headerUser)
-				r.Header.Del(headerTenant)
-			}
-			next.ServeHTTP(w, r)
-			return
-		}
-		if v == nil {
-			writeError(w, http.StatusUnauthorized, "nicht angemeldet")
-			return
-		}
-		claims, err := v.Verify(r.Context(), token)
-		if err != nil {
-			if errors.Is(err, moduleauth.ErrInvalidToken) {
-				writeError(w, http.StatusUnauthorized, "nicht angemeldet")
-			} else {
-				writeError(w, http.StatusServiceUnavailable, "identitaetsdienst nicht erreichbar")
-			}
-			return
-		}
-		r.Header.Set(headerUser, claims.Subject)
-		r.Header.Set(headerTenant, claims.Tenant)
-		if claims.Role != "" {
-			r.Header.Set(headerRole, claims.Role)
-		}
-		next.ServeHTTP(w, r)
-	})
-}
-
-func bearer(r *http.Request) (string, bool) {
-	h := r.Header.Get("Authorization")
-	if len(h) > 7 && strings.EqualFold(h[:7], "bearer ") {
-		return strings.TrimSpace(h[7:]), true
-	}
-	return "", false
-}
-
-func writeError(w http.ResponseWriter, status int, message string) {
-	w.Header().Set("Content-Type", "application/json")
-	w.WriteHeader(status)
-	_ = json.NewEncoder(w).Encode(map[string]string{"error": message})
-}

workflow/internal/bearerauth/bearerauth_test.go

@@ -1,75 +0,0 @@
-package bearerauth
-
-import (
-	"net/http"
-	"net/http/httptest"
-	"testing"
-
-	"gitea.perlbach24.de/scripte/nexarch/common/moduleauth"
-)
-
-func echo() http.Handler {
-	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		_, _ = w.Write([]byte(r.Header.Get("X-User-Id") + "|" + r.Header.Get("X-Tenant-Slug") + "|" + r.Header.Get("X-User-Role")))
-	})
-}
-
-func do(h http.Handler, hdr map[string]string) *httptest.ResponseRecorder {
-	req := httptest.NewRequest(http.MethodGet, "/x", nil)
-	for k, v := range hdr {
-		req.Header.Set(k, v)
-	}
-	rec := httptest.NewRecorder()
-	h.ServeHTTP(rec, req)
-	return rec
-}
-
-func TestBearerOverridesClientSuppliedHeaders(t *testing.T) {
-	v, iss := moduleauth.NewTestVerifierAndIssuer()
-	h := Middleware(v, false, echo())
-	rec := do(h, map[string]string{
-		"Authorization": "Bearer " + iss.Token("u1", "acme", "user", "", ""),
-		"X-User-Id":     "fremd", "X-Tenant-Slug": "fremd",
-	})
-	if rec.Code != 200 || rec.Body.String() != "u1|acme|user" {
-		t.Fatalf("code=%d body=%q", rec.Code, rec.Body.String())
-	}
-}
-
-func TestInvalidBearerIs401(t *testing.T) {
-	v, _ := moduleauth.NewTestVerifierAndIssuer()
-	_, other := moduleauth.NewTestVerifierAndIssuer()
-	h := Middleware(v, true, echo())
-	if rec := do(h, map[string]string{"Authorization": "Bearer " + other.Token("u1", "acme", "user", "", ""), "X-User-Id": "evil"}); rec.Code != 401 {
-		t.Fatalf("code=%d", rec.Code)
-	}
-	if rec := do(Middleware(nil, true, echo()), map[string]string{"Authorization": "Bearer x.y.z"}); rec.Code != 401 {
-		t.Fatalf("ohne Verifier code=%d", rec.Code)
-	}
-}
-
-func TestWithoutBearerHeadersStrippedUnlessAllowed(t *testing.T) {
-	v, _ := moduleauth.NewTestVerifierAndIssuer()
-	hdr := map[string]string{"X-User-Id": "u9", "X-Tenant-Slug": "t9"}
-	if rec := do(Middleware(v, false, echo()), hdr); rec.Body.String() != "||" {
-		t.Fatalf("Header muessen entfernt werden, bekam %q", rec.Body.String())
-	}
-	if rec := do(Middleware(v, true, echo()), hdr); rec.Body.String() != "u9|t9|" {
-		t.Fatalf("Uebergangsmodus muss Header durchlassen, bekam %q", rec.Body.String())
-	}
-}
-
-func TestRoleOnlyFromToken(t *testing.T) {
-	v, iss := moduleauth.NewTestVerifierAndIssuer()
-	h := Middleware(v, true, echo())
-	rec := do(h, map[string]string{
-		"Authorization": "Bearer " + iss.Token("u1", "acme", "tenant_admin", "", ""),
-		"X-User-Role":   "tenant_admin_fremd",
-	})
-	if rec.Body.String() != "u1|acme|tenant_admin" {
-		t.Fatalf("Rolle muss aus dem Token kommen, bekam %q", rec.Body.String())
-	}
-	if rec := do(h, map[string]string{"X-User-Role": "tenant_admin"}); rec.Body.String() != "||" {
-		t.Fatalf("Rolle ohne Token muss entfernt werden, bekam %q", rec.Body.String())
-	}
-}

workflow/internal/pflichttestgate/gate.go

@@ -18,7 +18,6 @@ import (
 // workflow/docs/TESTSTRATEGIE.md Abschnitt 4.
 var sensitivePathPatterns = []*regexp.Regexp{
 	regexp.MustCompile(`(^|/)workflow/internal/moduleadapter/`),
-	regexp.MustCompile(`(^|/)workflow/internal/bearerauth/`),
 	regexp.MustCompile(`(^|/)workflow/internal/backchannel/`),
 	regexp.MustCompile(`(^|/)workflow/internal/backchannelapi/`),
 	regexp.MustCompile(`(^|/)workflow/internal/dualcheck/`),

workflow/internal/pflichttestgate/gate_test.go

@@ -35,7 +35,6 @@ func TestCheckDiff_IgnoresNonSensitivePaths(t *testing.T) {
 func TestCheckDiff_CoversAllSensitivePackages(t *testing.T) {
 	files := []string{
 		"workflow/internal/moduleadapter/x.go",
-		"workflow/internal/bearerauth/x.go",
 		"workflow/internal/backchannel/x.go",
 		"workflow/internal/backchannelapi/x.go",
 		"workflow/internal/dualcheck/x.go",
@@ -46,10 +45,3 @@ func TestCheckDiff_CoversAllSensitivePackages(t *testing.T) {
 		t.Fatalf("erwartet %d verstoesse (je package einer), habe %d: %+v", len(files), len(violations), violations)
 	}
 }
-
-func TestCheckDiff_FlagsBearerAuthChangeWithoutTest(t *testing.T) {
-	violations := CheckDiff([]string{"workflow/internal/bearerauth/bearerauth.go"})
-	if len(violations) != 1 || violations[0].Package != "workflow/internal/bearerauth" {
-		t.Fatalf("erwartet 1 verstoss fuer bearerauth, habe: %+v", violations)
-	}
-}