IAM-05: ldap-active-directory-anbindung

internal/ldapsync: Client ist eine Schnittstelle (Search() liefert Entry-
Liste) — in dieser Umgebung steht kein echter LDAP/AD-Server zur Verfuegung
(analog AUD-05/Archive), daher ist die Sync-/Rollenzuordnungs-Logik
vollstaendig gegen einen Fake getestet, eine echte Verbindungspruefung gegen
LDAP/AD steht noch aus.

RoleMappingStore (Akzeptanzkriterium 3): explizite Erlaubnis-Matrix
LDAP-Gruppe -> Rolle. HighestRoleFor vergibt NUR fuer explizit gemappte
Gruppen eine Rolle — eine unbekannte Gruppe, auch wenn ihr Name zufaellig
wie eine interne Rolle aussieht ("tenant_admin", "superadmin"), traegt
strukturell nichts bei (kein Code-Pfad, der eine ungemappte Gruppe je einer
Rolle zuordnet) — behebt die aus archivmail bekannte Privilege-Escalation-
Fehlerklasse von Grund auf statt nachtraeglich zu haerten.

Syncer.Run ruft Search() als ALLERERSTES auf; schlaegt es fehl, wird ohne
jede Aenderung an bestehenden Konten abgebrochen (Akzeptanzkriterium 2).
Pro Eintrag isolierte Fehler landen in SyncResult.Failed, ohne andere
Eintraege zu beeintraechtigen. Deaktivierung in LDAP wird als
users.Deactivate uebernommen (Akzeptanzkriterium 3). ldap_config speichert
bewusst nur den NAMEN einer Umgebungsvariable fuer das Bind-Passwort, nie
das Passwort selbst.

internal/rbac (RBAC-01) wurde 1:1 aus dem rbac-01-Branch uebernommen (git
show aus derselben Repo-Historie) — IAM-05 haengt an RBAC-01 fuer die
Rollenzuweisung, beide Boards leben aber auf getrennten, noch nicht
gemergten Feature-Branches ohne gemeinsame Historie.

Pruefungen (ausgefuehrt auf root@192.168.1.131, go build/vet/test PASS):
1. Rollen-Erlaubnis-Matrix mit Positiv- und Negativfaellen automatisiert
   getestet (keine Privilege-Escalation) — TestRoleMapping_PositiveAndNegativeCases:
   gemappte Gruppe liefert Rolle, unbekannte Gruppe und rollen-aehnlich
   benannte, aber nie gemappte Gruppen liefern keine. PASS.
2. Synchronisationslauf mit fehlerhafter/nicht erreichbarer LDAP-Quelle
   bricht kontrolliert ab, ohne bestehende Konten zu beschaedigen —
   TestSyncer_AbortsCleanlyOnSourceError: bestehendes Konto bleibt nach
   fehlgeschlagenem Lauf unveraendert aktiv. PASS.
3. Deaktivierung eines Benutzers in LDAP wird bei naechster Synchronisation
   korrekt uebernommen — TestSyncer_AppliesDeactivationOnNextRun: erster
   Lauf legt aktiven Benutzer an, zweiter Lauf mit Disabled=true setzt ihn
   auf inaktiv. PASS.

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

DEVLOG.md

@@ -129,21 +129,3 @@ Keine Commits in dieser Session.
 - migrations/0001_tenant_registry.sql | 10 ++++++++++
 
 ---
-## 2026-08-28 22:48 – 22:51 (3m)
-**Beschreibung:** Claude Code Session
-**Projekt:** nexarch
-
-### Commits
-- c344dea TEN-08: tenant-loeschung-unter-retention-vorbehalt-gobd (RetentionChecker-Schnittstelle gegen Archive RET-03/CMP-06, ProcessDueDeletions haelt gesperrte Tenants zurueck)
-
-### Geänderte Dateien
-- internal/tenant/lifecycle.go                    |  45 ++++++++++++++++++++++++++++++++++++++++-----
-- internal/tenant/lifecycle_test.go               |   4 +++-
-- internal/tenant/registry.go                     |   7 +++++--
-- internal/tenant/retention.go                    |  34 ++++++++++++++++++++++++++++++++++
-- internal/tenant/retention_test.go               | 164 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
-- internal/tenant/tenant.go                       |   5 +++++
-- migrations/0004_tenant_retention_block.down.sql |   2 ++
-- migrations/0004_tenant_retention_block.up.sql   |   6 ++++++
-
----

docs/CODING-GUIDELINES-CORE.md

@@ -1,73 +0,0 @@
-# NEXARCH Core – Projektweite Sicherheits-Coding-Konventionen
-
-Stand: 2026-08-28. Ticket: IAM-15. Ergänzt `docs/TESTSTRATEGIE-CORE.md` (QA-01) um Coding-Regeln,
-die als Code-Review-Checkliste gelten — keine dieser Regeln ist optional oder "nur für ein Modul".
-
-## 1. Warum dieses Dokument existiert
-
-Eine sicherheitsrelevante Coding-Regel, die nur einmal an einer Stelle vorgemacht statt projektweit
-verankert wird, wird beim nächsten neuen Vergleich vergessen. Das gilt für jede Regel in diesem
-Dokument gleichermaßen — die erste Regel (SQL) ist bereits als Konvention etabliert, die zweite
-(timing-safe Vergleich, IAM-15) macht sie hier zum ersten Mal explizit schriftlich.
-
-## 2. Regel: Kein `fmt.Sprintf` für SQL-Bestandteile aus Nutzereingabe
-
-**Spalten-/Tabellennamen ausschließlich aus statischen Konstanten bzw. einem geschlossenen
-Enum/Switch-Typ, nie aus Nutzereingabe oder generischem String-Zusammenbau — auch nicht hinter
-einer Whitelist-Funktion.** Werte (nicht Bezeichner) gehören als Parameter (`$1`, `$2`, …) in die
-Query, niemals interpoliert.
-
-Lehre aus beiden Altsystemen (`known-issues-archivdms.md` Punkt 10, `known-issues-archivmail.md`
-Punkt 12): dynamische Tabellennamen via `fmt.Sprintf`, nur durch eine fragile Whitelist-Funktion
-abgesichert. Siehe DMS/Mail `SRC-11` für die board-spezifische Umsetzung dieser Regel im
-Suchindex-Kontext.
-
-**Referenzbeispiel (korrekt):** `internal/tenant/lifecycle.go`, `ProcessDueDeletions` — Statuswerte
-und IDs ausschließlich als Parameter (`$1`, `$2`, …), niemals interpoliert; der einzige Einsatz von
-`fmt.Sprintf` im Package baut einen **Datenbanknamen aus einem bereits validierten Slug**
-(`dbNameForSlug`, `slugPattern` in `tenant.go` erzwingt `^[a-z][a-z0-9_]{1,48}$` vor jeder
-Verwendung) — keine ungeprüfte Nutzereingabe erreicht die Query.
-
-## 3. Regel: Timing-safe Vergleich für jede sicherheitsrelevante Zugriffsentscheidung (IAM-15)
-
-**Jeder Vergleich, der eine sicherheitsrelevante Zugriffsentscheidung trifft — Passwort-Hash, Token,
-Signatur, 2FA-Code/-Wiederherstellungscode — nutzt einen timing-safe/constant-time Vergleich, nie
-den regulären `==`-Operator.** Ein naiver `==`-Vergleich zweier Byte-Folgen bricht bei der ersten
-abweichenden Stelle ab; die dadurch messbare Laufzeitdifferenz lässt sich aus der Ferne ausmessen und
-erlaubt ein Byte-für-Byte-Erraten des korrekten Werts (Timing-Angriff).
-
-**So wird es gemacht:** `internal/timingsafe` (dieses Ticket) bündelt die kanonische Implementierung
-(`crypto/subtle.ConstantTimeCompare`) für neue Vergleichsstellen:
-
-```go
-import "gitea.perlbach24.de/scripte/nexarch/internal/timingsafe"
-
-if !timingsafe.EqualString(providedCode, expectedCode) {
-    return ErrInvalid
-}
-```
-
-Ausnahme: `bcrypt.CompareHashAndPassword` (Passwort-Hashes) ist bereits von Haus aus timing-safe —
-hier ist kein zusätzlicher Wrapper nötig.
-
-### 3.1 Audit bestehender Vergleichsstellen (Prüfung 2)
-
-Durchgeführt 2026-08-28, Ergebnis: **alle bestehenden sicherheitsrelevanten Vergleichsstellen
-implementierten die Regel bereits korrekt**, unabhängig voneinander mit `crypto/subtle` — nichts
-musste korrigiert werden (Akzeptanzkriterium 3, „ggf.").
-
-| Ort | Was wird verglichen | Fundstelle |
-|---|---|---|
-| `internal/totp/totp.go`, `Validate` | TOTP-Code (2FA) | nutzte bereits `subtle.ConstantTimeCompare` direkt, in diesem Ticket auf `timingsafe.EqualString` umgestellt (erster Verwender des neuen Packages) |
-| `internal/webhook/dispatcher.go`, `VerifySignature` | HMAC-Webhook-Signatur | `subtle.ConstantTimeCompare(expectedBytes, gotBytes)` |
-| `internal/moduleregistry/credentials.go`, `Authenticate` | Service-Credential-Secret-Hash | eigene `timingSafeEqual`-Hilfsfunktion, gleiches Muster |
-| `internal/authtoken/token.go`, `Consume` (Passwort-Reset/Einladung) | Einmal-Token | Hash-Lookup über DB-Index (`WHERE token_hash = $1`), kein manueller Byte-Vergleich nötig — bei zufälligen, hochentropischen Token ist der indexierte Hash-Abgleich gleichwertig sicher |
-
-Neue Vergleichsstellen sollen `internal/timingsafe` verwenden, statt das Muster erneut inline zu
-duplizieren — bestehende Stellen müssen dafür nicht umgebaut werden (kein Umbau angrenzender
-Bereiche über Board-Branch-Grenzen hinweg).
-
-## 4. Wie diese Liste wächst
-
-Neue projektweite Sicherheits-Coding-Regeln werden hier ergänzt, sobald sie (wie SQL-Sprintf und
-timing-safe Vergleich) mehr als einmal unabhängig als Lehre auftauchen — nicht vorab spekulativ.

internal/ldapsync/client.go

@@ -0,0 +1,27 @@
+// Package ldapsync implementiert Core IAM-05: LDAP/Active-Directory-
+// Anbindung je Tenant fuer Benutzer-Synchronisation, Login-Delegation und
+// Rollenzuordnung aus Gruppen ueber eine explizite Erlaubnis-Matrix.
+//
+// WICHTIGER HINWEIS: In dieser Umgebung steht kein echter LDAP/AD-Server zur
+// Verfuegung (analog zur Archive-Engine bei AUD-05). Client ist deshalb eine
+// Schnittstelle — die Sync-/Rollenzuordnungs-Logik ist gegen einen Fake in
+// den Tests vollstaendig geprueft, eine echte Verbindungspruefung gegen
+// LDAP/AD steht noch aus.
+package ldapsync
+
+// Entry ist ein aus LDAP gelesener Benutzerdatensatz.
+type Entry struct {
+	Email    string
+	Name     string
+	Groups   []string
+	Disabled bool
+}
+
+// Client abstrahiert die eigentliche LDAP-Verbindung (Bind + Search).
+type Client interface {
+	// Search liefert alle Benutzer, die dem konfigurierten Filter
+	// entsprechen. Ein Fehler bedeutet "Quelle nicht erreichbar oder
+	// fehlerhaft" — Run() bricht dann komplett und folgenlos ab
+	// (Akzeptanzkriterium 2).
+	Search() ([]Entry, error)
+}

internal/ldapsync/ldapsync_test.go

@@ -0,0 +1,204 @@
+package ldapsync
+
+import (
+	"context"
+	"errors"
+	"fmt"
+	"os"
+	"testing"
+	"time"
+
+	"github.com/jackc/pgx/v5/pgxpool"
+
+	"gitea.perlbach24.de/scripte/nexarch/internal/rbac"
+	"gitea.perlbach24.de/scripte/nexarch/internal/user"
+)
+
+type fakeClient struct {
+	entries []Entry
+	err     error
+}
+
+func (f *fakeClient) Search() ([]Entry, error) {
+	if f.err != nil {
+		return nil, f.err
+	}
+	return f.entries, nil
+}
+
+func setupTest(t *testing.T) (*user.TenantUserStore, *rbac.Store, *RoleMappingStore, func()) {
+	t.Helper()
+	adminDSN := os.Getenv("TEST_ADMIN_DSN")
+	if adminDSN == "" {
+		t.Skip("TEST_ADMIN_DSN nicht gesetzt, Integrationstest uebersprungen")
+	}
+	ctx := context.Background()
+
+	pool, err := pgxpool.New(ctx, adminDSN)
+	if err != nil {
+		t.Fatalf("pool: %v", err)
+	}
+	if _, err := pool.Exec(ctx, `
+		CREATE TABLE IF NOT EXISTS users (
+			id UUID PRIMARY KEY DEFAULT gen_random_uuid(), email TEXT NOT NULL UNIQUE, name TEXT NOT NULL,
+			status TEXT NOT NULL DEFAULT 'active', password_hash TEXT NOT NULL DEFAULT '',
+			created_at TIMESTAMPTZ NOT NULL DEFAULT now(), updated_at TIMESTAMPTZ NOT NULL DEFAULT now()
+		);
+		CREATE TABLE IF NOT EXISTS role_assignments (
+			user_id UUID PRIMARY KEY REFERENCES users(id), role TEXT NOT NULL CHECK (role IN ('user','tenant_admin')),
+			granted_by TEXT NOT NULL, granted_at TIMESTAMPTZ NOT NULL DEFAULT now()
+		);
+		CREATE TABLE IF NOT EXISTS role_assignment_history (
+			id UUID PRIMARY KEY DEFAULT gen_random_uuid(), user_id UUID NOT NULL, role TEXT NOT NULL,
+			granted_by TEXT NOT NULL, granted_at TIMESTAMPTZ NOT NULL DEFAULT now()
+		);
+		CREATE TABLE IF NOT EXISTS ldap_group_role_mapping (
+			ldap_group TEXT PRIMARY KEY, role TEXT NOT NULL CHECK (role IN ('user','tenant_admin'))
+		);
+	`); err != nil {
+		t.Fatalf("schema: %v", err)
+	}
+
+	cleanup := func() { pool.Close() }
+	return user.NewTenantUserStore(pool), rbac.NewStore(pool), NewRoleMappingStore(pool), cleanup
+}
+
+func uniqueEmail() string {
+	return fmt.Sprintf("ldap-test-%d@example.com", time.Now().UnixNano())
+}
+
+// Akzeptanzkriterium 3 + Pruefung 1: Rollen-Erlaubnis-Matrix mit Positiv- und
+// Negativfaellen — keine Privilege-Escalation ueber Gruppenmitgliedschaft.
+func TestRoleMapping_PositiveAndNegativeCases(t *testing.T) {
+	_, _, mappings, cleanup := setupTest(t)
+	defer cleanup()
+	ctx := context.Background()
+
+	if err := mappings.SetMapping(ctx, "cn=IT-Admins,ou=groups,dc=example,dc=com", rbac.RoleTenantAdmin); err != nil {
+		t.Fatalf("set mapping: %v", err)
+	}
+
+	// Positivfall: gemappte Gruppe liefert die konfigurierte Rolle.
+	role, ok, err := mappings.HighestRoleFor(ctx, []string{"cn=IT-Admins,ou=groups,dc=example,dc=com"})
+	if err != nil {
+		t.Fatalf("highest role: %v", err)
+	}
+	if !ok || role != rbac.RoleTenantAdmin {
+		t.Fatalf("erwartet tenant_admin, habe ok=%v role=%q", ok, role)
+	}
+
+	// Negativfall 1: unbekannte Gruppe vergibt keine Rolle.
+	_, ok, err = mappings.HighestRoleFor(ctx, []string{"cn=Praktikanten,ou=groups,dc=example,dc=com"})
+	if err != nil {
+		t.Fatalf("highest role (unbekannt): %v", err)
+	}
+	if ok {
+		t.Fatal("erwartet KEINE rolle fuer nicht gemappte gruppe")
+	}
+
+	// Negativfall 2 (Privilege-Escalation-Versuch): eine Gruppe, die zufaellig
+	// wie eine interne Rolle heisst, aber NIE explizit gemappt wurde, darf
+	// keine Rolle vergeben.
+	_, ok, err = mappings.HighestRoleFor(ctx, []string{"tenant_admin", "superadmin"})
+	if err != nil {
+		t.Fatalf("highest role (namens-trick): %v", err)
+	}
+	if ok {
+		t.Fatal("erwartet KEINE privilege-escalation ueber gruppennamen, die wie rollen aussehen")
+	}
+}
+
+// Akzeptanzkriterium 1: Synchronisationslauf legt Benutzer an.
+func TestSyncer_CreatesUsers(t *testing.T) {
+	users, roles, mappings, cleanup := setupTest(t)
+	defer cleanup()
+	ctx := context.Background()
+
+	email := uniqueEmail()
+	client := &fakeClient{entries: []Entry{{Email: email, Name: "LDAP Nutzer"}}}
+	syncer := NewSyncer(client, users, roles, mappings)
+
+	result, err := syncer.Run(ctx)
+	if err != nil {
+		t.Fatalf("run: %v", err)
+	}
+	if result.Created != 1 {
+		t.Fatalf("erwartet 1 angelegten benutzer, habe %d", result.Created)
+	}
+
+	if _, err := users.GetByEmailForAuth(ctx, email); err != nil {
+		t.Fatalf("benutzer sollte existieren: %v", err)
+	}
+}
+
+// Akzeptanzkriterium 2 + Pruefung 2: fehlerhafte/nicht erreichbare LDAP-
+// Quelle bricht kontrolliert ab, ohne bestehende Konten zu beschaedigen.
+func TestSyncer_AbortsCleanlyOnSourceError(t *testing.T) {
+	users, roles, mappings, cleanup := setupTest(t)
+	defer cleanup()
+	ctx := context.Background()
+
+	// Ein bereits bestehendes Konto, das UNBERUEHRT bleiben muss.
+	existingEmail := uniqueEmail()
+	existing, err := users.Create(ctx, existingEmail, "Bestehender Nutzer")
+	if err != nil {
+		t.Fatalf("create existing: %v", err)
+	}
+
+	client := &fakeClient{err: errors.New("ldap-server nicht erreichbar (simuliert)")}
+	syncer := NewSyncer(client, users, roles, mappings)
+
+	if _, err := syncer.Run(ctx); err == nil {
+		t.Fatal("erwartet fehler bei nicht erreichbarer ldap-quelle")
+	}
+
+	// Bestehendes Konto unveraendert (noch aktiv, keine geloeschten/neuen Zeilen).
+	got, err := users.Get(ctx, existing.ID)
+	if err != nil {
+		t.Fatalf("bestehendes konto sollte weiterhin existieren: %v", err)
+	}
+	if got.Status != user.StatusActive {
+		t.Fatalf("bestehendes konto sollte unveraendert aktiv sein, ist %q", got.Status)
+	}
+}
+
+// Akzeptanzkriterium 3 + Pruefung 3: Deaktivierung in LDAP wird bei
+// naechster Synchronisation korrekt uebernommen.
+func TestSyncer_AppliesDeactivationOnNextRun(t *testing.T) {
+	users, roles, mappings, cleanup := setupTest(t)
+	defer cleanup()
+	ctx := context.Background()
+
+	email := uniqueEmail()
+	client := &fakeClient{entries: []Entry{{Email: email, Name: "Wird Deaktiviert"}}}
+	syncer := NewSyncer(client, users, roles, mappings)
+
+	if _, err := syncer.Run(ctx); err != nil {
+		t.Fatalf("erster lauf: %v", err)
+	}
+	u, err := users.GetByEmailForAuth(ctx, email)
+	if err != nil {
+		t.Fatalf("benutzer nach erstem lauf: %v", err)
+	}
+	if u.User.Status != user.StatusActive {
+		t.Fatalf("erwartet aktiv nach erstem lauf, habe %q", u.User.Status)
+	}
+
+	// Zweiter Lauf: derselbe Benutzer ist jetzt in LDAP deaktiviert.
+	client.entries[0].Disabled = true
+	result, err := syncer.Run(ctx)
+	if err != nil {
+		t.Fatalf("zweiter lauf: %v", err)
+	}
+	if result.Deactivated != 1 {
+		t.Fatalf("erwartet 1 deaktivierung, habe %d", result.Deactivated)
+	}
+
+	u, err = users.GetByEmailForAuth(ctx, email)
+	if err != nil {
+		t.Fatalf("benutzer nach zweitem lauf: %v", err)
+	}
+	if u.User.Status != user.StatusInactive {
+		t.Fatalf("erwartet inaktiv nach deaktivierung in ldap, habe %q", u.User.Status)
+	}
+}

internal/ldapsync/role_mapping.go

@@ -0,0 +1,79 @@
+package ldapsync
+
+import (
+	"context"
+	"errors"
+	"fmt"
+
+	"github.com/jackc/pgx/v5"
+	"github.com/jackc/pgx/v5/pgxpool"
+
+	"gitea.perlbach24.de/scripte/nexarch/internal/rbac"
+)
+
+// RoleMappingStore verwaltet die Erlaubnis-Matrix LDAP-Gruppe -> Rolle
+// (Akzeptanzkriterium 3). Nur explizit gemappte Gruppen vergeben eine Rolle
+// — der Schutz gegen Privilege-Escalation ist strukturell: es gibt keinen
+// Code-Pfad, der eine unbekannte Gruppe irgendeiner Rolle zuordnet.
+type RoleMappingStore struct {
+	pool *pgxpool.Pool
+}
+
+func NewRoleMappingStore(pool *pgxpool.Pool) *RoleMappingStore {
+	return &RoleMappingStore{pool: pool}
+}
+
+// SetMapping ordnet eine LDAP-Gruppe genau einer NEXARCH-Rolle zu. Nur
+// 'user'/'tenant_admin' sind zuweisbar (dieselbe Matrix wie RBAC-01/03) —
+// 'superadmin' bleibt mandantenuebergreifend und nie ueber LDAP erreichbar.
+func (s *RoleMappingStore) SetMapping(ctx context.Context, ldapGroup string, role rbac.Role) error {
+	if role != rbac.RoleUser && role != rbac.RoleTenantAdmin {
+		return fmt.Errorf("ldapsync: rolle %q ist ueber ldap nicht zuweisbar", role)
+	}
+	_, err := s.pool.Exec(ctx, `
+		INSERT INTO ldap_group_role_mapping (ldap_group, role) VALUES ($1, $2)
+		ON CONFLICT (ldap_group) DO UPDATE SET role = $2
+	`, ldapGroup, string(role))
+	if err != nil {
+		return fmt.Errorf("gruppenzuordnung speichern: %w", err)
+	}
+	return nil
+}
+
+// roleFor liefert die einer LDAP-Gruppe zugeordnete Rolle, sofern konfiguriert.
+func (s *RoleMappingStore) roleFor(ctx context.Context, ldapGroup string) (rbac.Role, bool, error) {
+	var role string
+	err := s.pool.QueryRow(ctx, `SELECT role FROM ldap_group_role_mapping WHERE ldap_group = $1`, ldapGroup).Scan(&role)
+	if err != nil {
+		if errors.Is(err, pgx.ErrNoRows) {
+			return "", false, nil
+		}
+		return "", false, fmt.Errorf("gruppenzuordnung lesen: %w", err)
+	}
+	return rbac.Role(role), true, nil
+}
+
+// HighestRoleFor bestimmt aus einer Liste von LDAP-Gruppen die hoechste
+// zugeordnete Rolle (tenant_admin > user). Gruppen ohne Eintrag in der
+// Matrix tragen NICHTS bei — auch nicht, wenn ihr Name zufaellig wie eine
+// interne Rolle aussieht (Akzeptanzkriterium 3 / Pruefung 1: keine
+// Privilege-Escalation ueber Gruppenmitgliedschaft).
+func (s *RoleMappingStore) HighestRoleFor(ctx context.Context, ldapGroups []string) (rbac.Role, bool, error) {
+	var best rbac.Role
+	found := false
+
+	for _, g := range ldapGroups {
+		role, ok, err := s.roleFor(ctx, g)
+		if err != nil {
+			return "", false, err
+		}
+		if !ok {
+			continue
+		}
+		if !found || role == rbac.RoleTenantAdmin {
+			best = role
+			found = true
+		}
+	}
+	return best, found, nil
+}

internal/ldapsync/sync.go

@@ -0,0 +1,93 @@
+package ldapsync
+
+import (
+	"context"
+	"errors"
+	"fmt"
+
+	"gitea.perlbach24.de/scripte/nexarch/internal/rbac"
+	"gitea.perlbach24.de/scripte/nexarch/internal/user"
+)
+
+// SyncResult fasst das Ergebnis eines Synchronisationslaufs zusammen.
+type SyncResult struct {
+	Created     int
+	Updated     int
+	Deactivated int
+	Failed      []string // Email-Adressen, bei denen die Verarbeitung fehlschlug
+}
+
+// Syncer fuehrt die Benutzer-/Rollen-Synchronisation aus LDAP aus.
+type Syncer struct {
+	client   Client
+	users    *user.TenantUserStore
+	roles    *rbac.Store
+	mappings *RoleMappingStore
+}
+
+func NewSyncer(client Client, users *user.TenantUserStore, roles *rbac.Store, mappings *RoleMappingStore) *Syncer {
+	return &Syncer{client: client, users: users, roles: roles, mappings: mappings}
+}
+
+// Run synchronisiert Benutzer aus LDAP (Akzeptanzkriterium 1). Ist die
+// LDAP-Quelle nicht erreichbar/fehlerhaft, wird SOFORT und OHNE jede
+// Aenderung an bestehenden Konten abgebrochen (Akzeptanzkriterium 2 /
+// Pruefung 2) — Search() wird als allererstes aufgerufen, bevor irgendein
+// Schreibzugriff stattfindet.
+func (s *Syncer) Run(ctx context.Context) (SyncResult, error) {
+	entries, err := s.client.Search()
+	if err != nil {
+		return SyncResult{}, fmt.Errorf("ldap-quelle nicht erreichbar, synchronisation abgebrochen: %w", err)
+	}
+
+	var result SyncResult
+	for _, entry := range entries {
+		if err := s.syncOne(ctx, entry, &result); err != nil {
+			// Ein fehlerhafter Einzeleintrag isoliert sich selbst — er
+			// beschaedigt weder bereits verarbeitete noch nachfolgende
+			// Konten (Akzeptanzkriterium 2).
+			result.Failed = append(result.Failed, entry.Email)
+		}
+	}
+	return result, nil
+}
+
+func (s *Syncer) syncOne(ctx context.Context, entry Entry, result *SyncResult) error {
+	existing, err := s.users.GetByEmailForAuth(ctx, entry.Email)
+	isNew := errors.Is(err, user.ErrNotFound)
+	if err != nil && !isNew {
+		return err
+	}
+
+	var userID string
+	if isNew {
+		created, err := s.users.Create(ctx, entry.Email, entry.Name)
+		if err != nil {
+			return err
+		}
+		userID = created.ID
+		result.Created++
+	} else {
+		userID = existing.User.ID
+		result.Updated++
+	}
+
+	// Deaktivierung in LDAP wird uebernommen (Akzeptanzkriterium 3 / Pruefung 3).
+	if entry.Disabled {
+		if _, err := s.users.Deactivate(ctx, userID); err != nil {
+			return err
+		}
+		result.Deactivated++
+	}
+
+	role, ok, err := s.mappings.HighestRoleFor(ctx, entry.Groups)
+	if err != nil {
+		return err
+	}
+	if ok {
+		if _, err := s.roles.Assign(ctx, userID, role, "ldap-sync"); err != nil {
+			return err
+		}
+	}
+	return nil
+}

internal/rbac/role.go

@@ -0,0 +1,71 @@
+// Package rbac implementiert Core RBAC-01: das Rollenmodell mit
+// Grundrollen/-rechten, Hierarchie und Zuweisung. Die eigentliche
+// Durchsetzung (erlaubt/verboten je Request) ist RBAC-02, dieses Paket
+// liefert nur Modell + Zuweisung (Casbin-Prinzip: Policy-Modell getrennt
+// von der Entscheidungsfunktion).
+package rbac
+
+// Role ist eine der drei Grundrollen. Superadmin lebt mandantenuebergreifend
+// (siehe internal/user.SuperadminStore aus IAM-01) und wird deshalb NICHT
+// ueber Store (tenant-gescoped) zugewiesen — die Existenz eines Superadmin-
+// Kontos IST die Rollenzuweisung.
+type Role string
+
+const (
+	RoleSuperadmin  Role = "superadmin"
+	RoleTenantAdmin Role = "tenant_admin"
+	RoleUser        Role = "user"
+)
+
+type Permission string
+
+const (
+	PermSelfRead       Permission = "self.read"
+	PermSelfUpdate     Permission = "self.update"
+	PermManageUsers    Permission = "tenant.manage_users"
+	PermManageSettings Permission = "tenant.manage_settings"
+	PermManageTenants  Permission = "platform.manage_tenants"
+)
+
+// parent bildet die Rollenhierarchie ab (Akzeptanzkriterium 2): eine Rolle
+// erbt alle Rechte ihrer Elternrolle. tenant_admin erbt von user,
+// superadmin erbt von tenant_admin.
+var parent = map[Role]Role{
+	RoleTenantAdmin: RoleUser,
+	RoleSuperadmin:  RoleTenantAdmin,
+}
+
+// direct sind die einer Rolle direkt (ohne Vererbung) zugeordneten Rechte.
+var direct = map[Role][]Permission{
+	RoleUser:        {PermSelfRead, PermSelfUpdate},
+	RoleTenantAdmin: {PermManageUsers, PermManageSettings},
+	RoleSuperadmin:  {PermManageTenants},
+}
+
+// EffectivePermissions liefert die vollstaendige, ueber die Hierarchie
+// aufgeloeste Rechtemenge einer Rolle (Akzeptanzkriterium 2 / Pruefung 2).
+func EffectivePermissions(role Role) []Permission {
+	seen := make(map[Permission]bool)
+	var out []Permission
+
+	for r, ok := role, true; ok; r, ok = parent[r] {
+		for _, p := range direct[r] {
+			if !seen[p] {
+				seen[p] = true
+				out = append(out, p)
+			}
+		}
+	}
+	return out
+}
+
+// HasPermission prueft, ob eine Rolle (inklusive geerbter Rechte) ein
+// bestimmtes Recht besitzt.
+func HasPermission(role Role, perm Permission) bool {
+	for _, p := range EffectivePermissions(role) {
+		if p == perm {
+			return true
+		}
+	}
+	return false
+}

internal/rbac/store.go

@@ -0,0 +1,124 @@
+package rbac
+
+import (
+	"context"
+	"errors"
+	"fmt"
+
+	"github.com/jackc/pgx/v5"
+	"github.com/jackc/pgx/v5/pgxpool"
+)
+
+var (
+	// ErrRoleNotAssignableInTenantScope wird geliefert, wenn versucht wird,
+	// eine mandantenuebergreifende Rolle (superadmin) ueber den tenant-
+	// gescopten Store zu vergeben — die erlaubte Matrix laesst hier nur
+	// user/tenant_admin zu (Akzeptanzkriterium 1 / Pruefung 1).
+	ErrRoleNotAssignableInTenantScope = errors.New("rbac: rolle ist in diesem geltungsbereich nicht zuweisbar")
+	ErrNotFound                       = errors.New("rbac: keine rollenzuweisung gefunden")
+)
+
+// assignableRoles ist die erlaubte Matrix fuer Store (tenant-gescoped).
+var assignableRoles = map[Role]bool{
+	RoleUser:        true,
+	RoleTenantAdmin: true,
+}
+
+type Assignment struct {
+	UserID    string
+	Role      Role
+	GrantedBy string
+}
+
+// Store verwaltet Rollenzuweisungen innerhalb GENAU EINER Tenant-Datenbank —
+// analog zu internal/user.TenantUserStore (Modell C: der Pool bestimmt den
+// Tenant, keine tenant_id-Spalte noetig).
+type Store struct {
+	pool *pgxpool.Pool
+}
+
+func NewStore(pool *pgxpool.Pool) *Store {
+	return &Store{pool: pool}
+}
+
+// Assign vergibt eine Rolle an einen Benutzer. grantedBy identifiziert den
+// Akteur, der die Zuweisung vorgenommen hat (Akzeptanzkriterium 3). Jede
+// Zuweisung wird zusaetzlich in role_assignment_history festgehalten, auch
+// wenn sie eine vorherige Rolle ersetzt.
+func (s *Store) Assign(ctx context.Context, userID string, role Role, grantedBy string) (Assignment, error) {
+	if !assignableRoles[role] {
+		return Assignment{}, ErrRoleNotAssignableInTenantScope
+	}
+	if grantedBy == "" {
+		return Assignment{}, errors.New("rbac: grantedBy darf nicht leer sein")
+	}
+
+	tx, err := s.pool.Begin(ctx)
+	if err != nil {
+		return Assignment{}, fmt.Errorf("transaktion starten: %w", err)
+	}
+	defer func() { _ = tx.Rollback(ctx) }()
+
+	if _, err := tx.Exec(ctx, `
+		INSERT INTO role_assignments (user_id, role, granted_by, granted_at)
+		VALUES ($1, $2, $3, now())
+		ON CONFLICT (user_id) DO UPDATE SET role = $2, granted_by = $3, granted_at = now()
+	`, userID, string(role), grantedBy); err != nil {
+		return Assignment{}, fmt.Errorf("rolle zuweisen: %w", err)
+	}
+
+	if _, err := tx.Exec(ctx, `
+		INSERT INTO role_assignment_history (user_id, role, granted_by, granted_at)
+		VALUES ($1, $2, $3, now())
+	`, userID, string(role), grantedBy); err != nil {
+		return Assignment{}, fmt.Errorf("historie schreiben: %w", err)
+	}
+
+	if err := tx.Commit(ctx); err != nil {
+		return Assignment{}, fmt.Errorf("transaktion committen: %w", err)
+	}
+
+	return Assignment{UserID: userID, Role: role, GrantedBy: grantedBy}, nil
+}
+
+func (s *Store) Get(ctx context.Context, userID string) (Assignment, error) {
+	var a Assignment
+	var role string
+	a.UserID = userID
+	if err := s.pool.QueryRow(ctx, `
+		SELECT role, granted_by FROM role_assignments WHERE user_id = $1
+	`, userID).Scan(&role, &a.GrantedBy); err != nil {
+		if errors.Is(err, pgx.ErrNoRows) {
+			return Assignment{}, ErrNotFound
+		}
+		return Assignment{}, fmt.Errorf("rollenzuweisung lesen: %w", err)
+	}
+	a.Role = Role(role)
+	return a, nil
+}
+
+// History liefert jede Rollenzuweisung eines Benutzers in chronologischer
+// Reihenfolge — die Grundlage fuer "wer hat wann welche Rolle vergeben"
+// (Akzeptanzkriterium 3).
+func (s *Store) History(ctx context.Context, userID string) ([]Assignment, error) {
+	rows, err := s.pool.Query(ctx, `
+		SELECT role, granted_by FROM role_assignment_history
+		WHERE user_id = $1 ORDER BY granted_at
+	`, userID)
+	if err != nil {
+		return nil, fmt.Errorf("historie abfragen: %w", err)
+	}
+	defer rows.Close()
+
+	var out []Assignment
+	for rows.Next() {
+		var role string
+		a := Assignment{UserID: userID}
+		if err := rows.Scan(&role, &a.GrantedBy); err != nil {
+			return nil, fmt.Errorf("historieneintrag lesen: %w", err)
+		}
+		a.Role = Role(role)
+		out = append(out, a)
+	}
+	return out, rows.Err()
+}

internal/timingsafe/timingsafe.go

@@ -1,30 +0,0 @@
-// Package timingsafe stellt die kanonische Implementierung der projektweiten
-// Coding-Konvention aus IAM-15 bereit: jeder Vergleich, der eine
-// sicherheitsrelevante Zugriffsentscheidung trifft (Passwort-Hash, Token,
-// Signatur, 2FA-Wiederherstellungscode), nutzt einen timing-safe/constant-time
-// Vergleich, nie den regulaeren ==-Operator. Siehe docs/CODING-GUIDELINES-CORE.md.
-//
-// Bestehende Vergleichsstellen (internal/totp, internal/webhook,
-// internal/moduleregistry) implementieren dasselbe Muster bereits inline mit
-// crypto/subtle direkt — dieses Package buendelt es fuer neue Vergleichsstellen,
-// ersetzt die bestehenden nicht zwangsweise (kein Umbau angrenzender Bereiche).
-package timingsafe
-
-import "crypto/subtle"
-
-// Equal vergleicht zwei Byte-Slices timing-safe. Unterschiedliche Laenge gilt
-// als "nicht gleich", ohne dass die Laufzeit dabei die Laenge verraet, die
-// zum Ergebnis gefuehrt hat, mehr als durch den Laengenunterschied ohnehin
-// unvermeidbar waere.
-func Equal(a, b []byte) bool {
-	if len(a) != len(b) {
-		return false
-	}
-	return subtle.ConstantTimeCompare(a, b) == 1
-}
-
-// EqualString ist die String-Variante von Equal fuer den haeufigen Fall,
-// dass beide Seiten bereits als string vorliegen (z. B. TOTP-Codes).
-func EqualString(a, b string) bool {
-	return Equal([]byte(a), []byte(b))
-}

internal/timingsafe/timingsafe_test.go

@@ -1,36 +0,0 @@
-package timingsafe
-
-import "testing"
-
-func TestEqual_SameBytes(t *testing.T) {
-	if !Equal([]byte("geheimnis"), []byte("geheimnis")) {
-		t.Fatal("identische Byte-Slices sollten gleich sein")
-	}
-}
-
-func TestEqual_DifferentBytes(t *testing.T) {
-	if Equal([]byte("geheimnis"), []byte("anders123")) {
-		t.Fatal("unterschiedliche Byte-Slices sollten ungleich sein")
-	}
-}
-
-func TestEqual_DifferentLength(t *testing.T) {
-	if Equal([]byte("kurz"), []byte("laengererstring")) {
-		t.Fatal("unterschiedliche Laenge sollte immer ungleich sein")
-	}
-}
-
-func TestEqual_EmptyVsEmpty(t *testing.T) {
-	if !Equal([]byte(""), []byte("")) {
-		t.Fatal("zwei leere Slices sollten gleich sein")
-	}
-}
-
-func TestEqualString_MatchesEqual(t *testing.T) {
-	if !EqualString("abc123", "abc123") {
-		t.Fatal("identische Strings sollten gleich sein")
-	}
-	if EqualString("abc123", "xyz789") {
-		t.Fatal("unterschiedliche Strings sollten ungleich sein")
-	}
-}

internal/totp/login.go

@@ -1,42 +0,0 @@
-package totp
-
-import (
-	"context"
-	"errors"
-
-	"gitea.perlbach24.de/scripte/nexarch/internal/auth"
-	"gitea.perlbach24.de/scripte/nexarch/internal/user"
-)
-
-var ErrSecondFactorRequired = errors.New("totp: zweiter faktor erforderlich")
-
-// LoginWithTOTP komponiert IAM-02s LoginService mit der 2FA-Pruefung, ohne
-// LoginService selbst zu veraendern. Ist 2FA fuer den Benutzer aktiv, MUSS
-// totpCode gueltig sein (Akzeptanzkriterium 2) — ein leerer/falscher Code
-// wird zuverlaessig abgewiesen, auch bei korrektem Passwort.
-func LoginWithTOTP(ctx context.Context, users *user.TenantUserStore, totpStore *Store, login *auth.LoginService, email, password, totpCode string) (token string, err error) {
-	creds, err := users.GetByEmailForAuth(ctx, email)
-	if err != nil {
-		return "", auth.ErrInvalidCredentials
-	}
-
-	enabled, err := totpStore.IsEnabled(ctx, creds.User.ID)
-	if err != nil {
-		return "", err
-	}
-
-	if enabled {
-		if totpCode == "" {
-			return "", ErrSecondFactorRequired
-		}
-		ok, err := totpStore.VerifyLoginCode(ctx, creds.User.ID, totpCode)
-		if err != nil {
-			return "", err
-		}
-		if !ok {
-			return "", ErrSecondFactorRequired
-		}
-	}
-
-	return login.Login(ctx, email, password)
-}

internal/totp/store.go

@@ -1,163 +0,0 @@
-package totp
-
-import (
-	"context"
-	"crypto/rand"
-	"crypto/sha256"
-	"encoding/hex"
-	"errors"
-	"fmt"
-	"time"
-
-	"github.com/jackc/pgx/v5"
-	"github.com/jackc/pgx/v5/pgxpool"
-)
-
-var (
-	ErrInvalidCode      = errors.New("totp: code ungueltig")
-	ErrNotConfirmed     = errors.New("totp: einrichtung noch nicht bestaetigt")
-	ErrRecoveryCodeUsed = errors.New("totp: wiederherstellungscode bereits verwendet oder ungueltig")
-)
-
-const RecoveryCodeCount = 10
-
-type Store struct {
-	pool *pgxpool.Pool
-}
-
-func NewStore(pool *pgxpool.Pool) *Store {
-	return &Store{pool: pool}
-}
-
-// BeginSetup erzeugt ein neues Secret fuer einen Benutzer und speichert es
-// als NOCH NICHT bestaetigt — der Code muss vor Aktivierung einmal
-// verifiziert werden (Akzeptanzkriterium 1).
-func (s *Store) BeginSetup(ctx context.Context, userID, issuer, accountName string) (secret, provisioningURI string, err error) {
-	secret, err = GenerateSecret()
-	if err != nil {
-		return "", "", err
-	}
-
-	_, err = s.pool.Exec(ctx, `
-		INSERT INTO totp_credentials (user_id, secret, confirmed)
-		VALUES ($1, $2, false)
-		ON CONFLICT (user_id) DO UPDATE SET secret = $2, confirmed = false
-	`, userID, secret)
-	if err != nil {
-		return "", "", fmt.Errorf("totp-einrichtung speichern: %w", err)
-	}
-
-	return secret, ProvisioningURI(issuer, accountName, secret), nil
-}
-
-// ConfirmSetup verifiziert den ersten eingegebenen Code und aktiviert 2FA
-// erst dann (Akzeptanzkriterium 1). Bei Erfolg werden einmalig
-// Wiederherstellungscodes erzeugt und im Klartext zurueckgegeben — gespeichert
-// wird nur deren Hash.
-func (s *Store) ConfirmSetup(ctx context.Context, userID, code string) (recoveryCodes []string, err error) {
-	var secret string
-	if err := s.pool.QueryRow(ctx, `
-		SELECT secret FROM totp_credentials WHERE user_id = $1
-	`, userID).Scan(&secret); err != nil {
-		if errors.Is(err, pgx.ErrNoRows) {
-			return nil, ErrNotConfirmed
-		}
-		return nil, fmt.Errorf("secret lesen: %w", err)
-	}
-
-	ok, err := Validate(secret, code, time.Now())
-	if err != nil {
-		return nil, err
-	}
-	if !ok {
-		return nil, ErrInvalidCode
-	}
-
-	if _, err := s.pool.Exec(ctx, `UPDATE totp_credentials SET confirmed = true WHERE user_id = $1`, userID); err != nil {
-		return nil, fmt.Errorf("bestaetigung speichern: %w", err)
-	}
-
-	recoveryCodes, err = s.generateRecoveryCodes(ctx, userID)
-	if err != nil {
-		return nil, err
-	}
-	return recoveryCodes, nil
-}
-
-func (s *Store) generateRecoveryCodes(ctx context.Context, userID string) ([]string, error) {
-	codes := make([]string, RecoveryCodeCount)
-	for i := range codes {
-		buf := make([]byte, 8)
-		if _, err := rand.Read(buf); err != nil {
-			return nil, fmt.Errorf("wiederherstellungscode erzeugen: %w", err)
-		}
-		code := hex.EncodeToString(buf)
-		codes[i] = code
-
-		hash := hashRecoveryCode(code)
-		if _, err := s.pool.Exec(ctx, `
-			INSERT INTO totp_recovery_codes (user_id, code_hash) VALUES ($1, $2)
-		`, userID, hash); err != nil {
-			return nil, fmt.Errorf("wiederherstellungscode speichern: %w", err)
-		}
-	}
-	return codes, nil
-}
-
-// IsEnabled prueft, ob 2FA fuer einen Benutzer aktiv (bestaetigt) ist.
-func (s *Store) IsEnabled(ctx context.Context, userID string) (bool, error) {
-	var confirmed bool
-	err := s.pool.QueryRow(ctx, `SELECT confirmed FROM totp_credentials WHERE user_id = $1`, userID).Scan(&confirmed)
-	if err != nil {
-		if errors.Is(err, pgx.ErrNoRows) {
-			return false, nil
-		}
-		return false, fmt.Errorf("status lesen: %w", err)
-	}
-	return confirmed, nil
-}
-
-// VerifyLoginCode akzeptiert entweder einen gueltigen TOTP-Code oder einen
-// noch nicht verwendeten Wiederherstellungscode (Akzeptanzkriterium 2 + 3).
-func (s *Store) VerifyLoginCode(ctx context.Context, userID, code string) (bool, error) {
-	var secret string
-	var confirmed bool
-	err := s.pool.QueryRow(ctx, `SELECT secret, confirmed FROM totp_credentials WHERE user_id = $1`, userID).Scan(&secret, &confirmed)
-	if err != nil {
-		if errors.Is(err, pgx.ErrNoRows) || !confirmed {
-			return false, nil
-		}
-		return false, fmt.Errorf("secret lesen: %w", err)
-	}
-	if !confirmed {
-		return false, nil
-	}
-
-	if ok, err := Validate(secret, code, time.Now()); err != nil {
-		return false, err
-	} else if ok {
-		return true, nil
-	}
-
-	return s.consumeRecoveryCode(ctx, userID, code)
-}
-
-// consumeRecoveryCode markiert einen Wiederherstellungscode atomar als
-// verwendet (WHERE used_at IS NULL) — ein bereits verbrauchter Code kann
-// nicht doppelt eingeloest werden (Akzeptanzkriterium 3 / Pruefung 2).
-func (s *Store) consumeRecoveryCode(ctx context.Context, userID, code string) (bool, error) {
-	hash := hashRecoveryCode(code)
-	tag, err := s.pool.Exec(ctx, `
-		UPDATE totp_recovery_codes SET used_at = now()
-		WHERE user_id = $1 AND code_hash = $2 AND used_at IS NULL
-	`, userID, hash)
-	if err != nil {
-		return false, fmt.Errorf("wiederherstellungscode einloesen: %w", err)
-	}
-	return tag.RowsAffected() == 1, nil
-}
-
-func hashRecoveryCode(code string) []byte {
-	sum := sha256.Sum256([]byte(code))
-	return sum[:]
-}

internal/totp/store_test.go

@@ -1,185 +0,0 @@
-package totp
-
-import (
-	"context"
-	"errors"
-	"fmt"
-	"os"
-	"testing"
-	"time"
-
-	"github.com/jackc/pgx/v5/pgxpool"
-
-	"gitea.perlbach24.de/scripte/nexarch/internal/auth"
-	"gitea.perlbach24.de/scripte/nexarch/internal/user"
-)
-
-func setupTest(t *testing.T) (*Store, *user.TenantUserStore, string, func()) {
-	t.Helper()
-	adminDSN := os.Getenv("TEST_ADMIN_DSN")
-	if adminDSN == "" {
-		t.Skip("TEST_ADMIN_DSN nicht gesetzt, Integrationstest uebersprungen")
-	}
-	ctx := context.Background()
-
-	pool, err := pgxpool.New(ctx, adminDSN)
-	if err != nil {
-		t.Fatalf("pool: %v", err)
-	}
-	if _, err := pool.Exec(ctx, `
-		CREATE TABLE IF NOT EXISTS users (
-			id UUID PRIMARY KEY DEFAULT gen_random_uuid(), email TEXT NOT NULL UNIQUE, name TEXT NOT NULL,
-			status TEXT NOT NULL DEFAULT 'active', password_hash TEXT NOT NULL DEFAULT '',
-			created_at TIMESTAMPTZ NOT NULL DEFAULT now(), updated_at TIMESTAMPTZ NOT NULL DEFAULT now()
-		);
-		CREATE TABLE IF NOT EXISTS totp_credentials (
-			user_id UUID PRIMARY KEY REFERENCES users(id), secret TEXT NOT NULL,
-			confirmed BOOLEAN NOT NULL DEFAULT false, created_at TIMESTAMPTZ NOT NULL DEFAULT now()
-		);
-		CREATE TABLE IF NOT EXISTS totp_recovery_codes (
-			id UUID PRIMARY KEY DEFAULT gen_random_uuid(), user_id UUID NOT NULL REFERENCES users(id),
-			code_hash BYTEA NOT NULL UNIQUE, used_at TIMESTAMPTZ, created_at TIMESTAMPTZ NOT NULL DEFAULT now()
-		);
-	`); err != nil {
-		t.Fatalf("schema: %v", err)
-	}
-
-	userStore := user.NewTenantUserStore(pool)
-	email := fmt.Sprintf("totp-test-%d@example.com", time.Now().UnixNano())
-	u, err := userStore.Create(ctx, email, "TOTP Test")
-	if err != nil {
-		t.Fatalf("create user: %v", err)
-	}
-	hash, err := auth.HashPassword("korrektes-passwort")
-	if err != nil {
-		t.Fatalf("hash: %v", err)
-	}
-	if err := userStore.SetPasswordHash(ctx, u.ID, hash); err != nil {
-		t.Fatalf("set password: %v", err)
-	}
-
-	cleanup := func() { pool.Close() }
-	return NewStore(pool), userStore, u.ID, cleanup
-}
-
-// Akzeptanzkriterium 1: Einrichtung per QR-Code (URI), Bestaetigung vor Aktivierung.
-func TestBeginAndConfirmSetup(t *testing.T) {
-	store, _, userID, cleanup := setupTest(t)
-	defer cleanup()
-	ctx := context.Background()
-
-	secret, uri, err := store.BeginSetup(ctx, userID, "NEXARCH", "user@example.com")
-	if err != nil {
-		t.Fatalf("begin setup: %v", err)
-	}
-	if uri == "" {
-		t.Fatal("erwartet nicht-leere provisioning-uri")
-	}
-
-	enabled, err := store.IsEnabled(ctx, userID)
-	if err != nil {
-		t.Fatalf("is enabled (vor bestaetigung): %v", err)
-	}
-	if enabled {
-		t.Fatal("erwartet NICHT aktiviert vor bestaetigung")
-	}
-
-	// Falscher Code darf nicht aktivieren.
-	if _, err := store.ConfirmSetup(ctx, userID, "000000"); !errors.Is(err, ErrInvalidCode) && err == nil {
-		t.Fatalf("erwartet ablehnung bei falschem code, habe err=%v", err)
-	}
-
-	code, err := GenerateCode(secret, time.Now())
-	if err != nil {
-		t.Fatalf("generate code: %v", err)
-	}
-	recoveryCodes, err := store.ConfirmSetup(ctx, userID, code)
-	if err != nil {
-		t.Fatalf("confirm setup: %v", err)
-	}
-	if len(recoveryCodes) != RecoveryCodeCount {
-		t.Fatalf("erwartet %d wiederherstellungscodes, habe %d", RecoveryCodeCount, len(recoveryCodes))
-	}
-
-	enabled, err = store.IsEnabled(ctx, userID)
-	if err != nil {
-		t.Fatalf("is enabled (nach bestaetigung): %v", err)
-	}
-	if !enabled {
-		t.Fatal("erwartet aktiviert nach erfolgreicher bestaetigung")
-	}
-}
-
-// Akzeptanzkriterium 2 + Pruefung 1: Login ohne zweiten Faktor bei aktivem
-// 2FA wird zuverlaessig abgewiesen.
-func TestLoginWithTOTP_RequiresSecondFactorWhenEnabled(t *testing.T) {
-	store, userStore, userID, cleanup := setupTest(t)
-	defer cleanup()
-	ctx := context.Background()
-
-	secret, _, err := store.BeginSetup(ctx, userID, "NEXARCH", "user@example.com")
-	if err != nil {
-		t.Fatalf("begin setup: %v", err)
-	}
-	code, _ := GenerateCode(secret, time.Now())
-	if _, err := store.ConfirmSetup(ctx, userID, code); err != nil {
-		t.Fatalf("confirm setup: %v", err)
-	}
-
-	u, err := userStore.Get(ctx, userID)
-	if err != nil {
-		t.Fatalf("get user: %v", err)
-	}
-	issuer := auth.NewTokenIssuer("test-secret-nur-fuer-tests")
-	loginService := auth.NewLoginService(userStore, issuer, "acme")
-
-	// Korrektes Passwort, aber KEIN 2FA-Code -> abgewiesen.
-	if _, err := LoginWithTOTP(ctx, userStore, store, loginService, u.Email, "korrektes-passwort", ""); !errors.Is(err, ErrSecondFactorRequired) {
-		t.Fatalf("erwartet ErrSecondFactorRequired ohne code, habe %v", err)
-	}
-
-	// Korrektes Passwort + korrekter aktueller Code -> erfolgreich.
-	loginCode, _ := GenerateCode(secret, time.Now())
-	token, err := LoginWithTOTP(ctx, userStore, store, loginService, u.Email, "korrektes-passwort", loginCode)
-	if err != nil {
-		t.Fatalf("login mit gueltigem 2fa-code: %v", err)
-	}
-	if token == "" {
-		t.Fatal("erwartet gueltiges token")
-	}
-}
-
-// Akzeptanzkriterium 3 + Pruefung 2: Wiederherstellungscode nach Nutzung als
-// verbraucht getestet.
-func TestVerifyLoginCode_RecoveryCodeIsSingleUse(t *testing.T) {
-	store, _, userID, cleanup := setupTest(t)
-	defer cleanup()
-	ctx := context.Background()
-
-	secret, _, err := store.BeginSetup(ctx, userID, "NEXARCH", "user@example.com")
-	if err != nil {
-		t.Fatalf("begin setup: %v", err)
-	}
-	code, _ := GenerateCode(secret, time.Now())
-	recoveryCodes, err := store.ConfirmSetup(ctx, userID, code)
-	if err != nil {
-		t.Fatalf("confirm setup: %v", err)
-	}
-	recovery := recoveryCodes[0]
-
-	ok, err := store.VerifyLoginCode(ctx, userID, recovery)
-	if err != nil {
-		t.Fatalf("verify (erste nutzung): %v", err)
-	}
-	if !ok {
-		t.Fatal("erwartet erfolgreiche erste nutzung des wiederherstellungscodes")
-	}
-
-	ok, err = store.VerifyLoginCode(ctx, userID, recovery)
-	if err != nil {
-		t.Fatalf("verify (zweite nutzung): %v", err)
-	}
-	if ok {
-		t.Fatal("erwartet abgelehnt bei erneuter nutzung desselben wiederherstellungscodes")
-	}
-}

internal/totp/totp.go

@@ -1,105 +0,0 @@
-// Package totp implementiert Core IAM-04: TOTP-basierte Zwei-Faktor-
-// Authentifizierung (RFC 6238, aufbauend auf HOTP RFC 4226) mit stdlib
-// crypto/hmac+sha1 — kein externes Modul noetig.
-package totp
-
-import (
-	"crypto/hmac"
-	"crypto/rand"
-	"crypto/sha1"
-	"encoding/base32"
-	"encoding/binary"
-	"fmt"
-	"net/url"
-	"time"
-
-	"gitea.perlbach24.de/scripte/nexarch/internal/timingsafe"
-)
-
-// StepSeconds ist das TOTP-Zeitfenster (RFC-6238-Standard: 30 Sekunden).
-const StepSeconds = 30
-
-// DefaultSkewSteps ist die dokumentierte Zeitversatz-Toleranz
-// (Akzeptanzkriterium/Pruefung 3): +/- 1 Schritt = +/- 30 Sekunden um die
-// aktuelle Serverzeit, insgesamt ein 90-Sekunden-Fenster. Grosszuegig genug
-// fuer leicht abweichende Client-Uhren, eng genug um ein TOTP-Fenster nicht
-// unbrauchbar lange gueltig zu lassen.
-const DefaultSkewSteps = 1
-
-// GenerateSecret erzeugt ein neues, zufaelliges TOTP-Secret (20 Byte, wie von
-// Authenticator-Apps erwartet), Base32-kodiert ohne Padding.
-func GenerateSecret() (string, error) {
-	buf := make([]byte, 20)
-	if _, err := rand.Read(buf); err != nil {
-		return "", fmt.Errorf("secret erzeugen: %w", err)
-	}
-	return base32.StdEncoding.WithPadding(base32.NoPadding).EncodeToString(buf), nil
-}
-
-// ProvisioningURI liefert die otpauth://-URI fuer den QR-Code
-// (Akzeptanzkriterium 1) — das eigentliche QR-Rendering ist Frontend-Sache
-// (IAM-08), hier wird nur die standardkonforme URI erzeugt.
-func ProvisioningURI(issuer, accountName, secret string) string {
-	label := url.PathEscape(issuer) + ":" + url.PathEscape(accountName)
-	q := url.Values{}
-	q.Set("secret", secret)
-	q.Set("issuer", issuer)
-	q.Set("algorithm", "SHA1")
-	q.Set("digits", "6")
-	q.Set("period", fmt.Sprintf("%d", StepSeconds))
-	return fmt.Sprintf("otpauth://totp/%s?%s", label, q.Encode())
-}
-
-// GenerateCode berechnet den 6-stelligen TOTP-Code fuer secret zum Zeitpunkt t.
-func GenerateCode(secret string, t time.Time) (string, error) {
-	key, err := decodeSecret(secret)
-	if err != nil {
-		return "", err
-	}
-	counter := uint64(t.Unix() / StepSeconds)
-	return hotp(key, counter), nil
-}
-
-// Validate prueft code gegen secret mit der dokumentierten Zeitversatz-
-// Toleranz (DefaultSkewSteps) um t. Vergleich ist timing-safe.
-func Validate(secret, code string, t time.Time) (bool, error) {
-	key, err := decodeSecret(secret)
-	if err != nil {
-		return false, err
-	}
-	counter := int64(t.Unix() / StepSeconds)
-
-	for delta := -DefaultSkewSteps; delta <= DefaultSkewSteps; delta++ {
-		candidate := hotp(key, uint64(counter+int64(delta)))
-		if timingsafe.EqualString(candidate, code) {
-			return true, nil
-		}
-	}
-	return false, nil
-}
-
-func decodeSecret(secret string) ([]byte, error) {
-	key, err := base32.StdEncoding.WithPadding(base32.NoPadding).DecodeString(secret)
-	if err != nil {
-		return nil, fmt.Errorf("secret dekodieren: %w", err)
-	}
-	return key, nil
-}
-
-// hotp implementiert RFC 4226 mit HMAC-SHA1, 6 Ziffern.
-func hotp(key []byte, counter uint64) string {
-	buf := make([]byte, 8)
-	binary.BigEndian.PutUint64(buf, counter)
-
-	mac := hmac.New(sha1.New, key)
-	mac.Write(buf)
-	sum := mac.Sum(nil)
-
-	offset := sum[len(sum)-1] & 0x0f
-	code := (uint32(sum[offset]&0x7f) << 24) |
-		(uint32(sum[offset+1]) << 16) |
-		(uint32(sum[offset+2]) << 8) |
-		uint32(sum[offset+3])
-
-	return fmt.Sprintf("%06d", code%1000000)
-}

internal/totp/totp_test.go

@@ -1,79 +0,0 @@
-package totp
-
-import (
-	"testing"
-	"time"
-)
-
-func TestGenerateAndValidateCode_RoundTrip(t *testing.T) {
-	secret, err := GenerateSecret()
-	if err != nil {
-		t.Fatalf("generate secret: %v", err)
-	}
-	now := time.Now()
-	code, err := GenerateCode(secret, now)
-	if err != nil {
-		t.Fatalf("generate code: %v", err)
-	}
-	if len(code) != 6 {
-		t.Fatalf("erwartet 6-stelligen code, habe %q", code)
-	}
-
-	ok, err := Validate(secret, code, now)
-	if err != nil {
-		t.Fatalf("validate: %v", err)
-	}
-	if !ok {
-		t.Fatal("erwartet gueltigen code")
-	}
-
-	wrongCode := "000000"
-	if code == wrongCode {
-		wrongCode = "111111"
-	}
-	ok, err = Validate(secret, wrongCode, now)
-	if err != nil {
-		t.Fatalf("validate (falsch): %v", err)
-	}
-	if ok {
-		t.Fatal("erwartet ungueltig fuer falschen code")
-	}
-}
-
-// Akzeptanzkriterium/Pruefung 3: Zeitversatz-Toleranz dokumentiert (siehe
-// DefaultSkewSteps = 1 Schritt = 30s) und getestet.
-func TestValidate_ClockSkewTolerance(t *testing.T) {
-	secret, err := GenerateSecret()
-	if err != nil {
-		t.Fatalf("generate secret: %v", err)
-	}
-	now := time.Now()
-
-	// Code aus dem VORHERIGEN Zeitfenster (bis zu 30s in der Vergangenheit)
-	// muss innerhalb der dokumentierten Toleranz noch akzeptiert werden.
-	pastCode, err := GenerateCode(secret, now.Add(-25*time.Second))
-	if err != nil {
-		t.Fatalf("generate code (past): %v", err)
-	}
-	ok, err := Validate(secret, pastCode, now)
-	if err != nil {
-		t.Fatalf("validate (past): %v", err)
-	}
-	if !ok {
-		t.Fatal("erwartet gueltig innerhalb der dokumentierten skew-toleranz (25s)")
-	}
-
-	// Code weit ausserhalb der Toleranz (mehrere Schritte in der
-	// Vergangenheit) muss abgelehnt werden.
-	farPastCode, err := GenerateCode(secret, now.Add(-5*time.Minute))
-	if err != nil {
-		t.Fatalf("generate code (far past): %v", err)
-	}
-	ok, err = Validate(secret, farPastCode, now)
-	if err != nil {
-		t.Fatalf("validate (far past): %v", err)
-	}
-	if ok {
-		t.Fatal("erwartet ungueltig weit ausserhalb der skew-toleranz (5min)")
-	}
-}

migrations/tenant/0002_role_assignments.down.sql

@@ -0,0 +1,2 @@
+DROP TABLE IF EXISTS role_assignment_history;
+DROP TABLE IF EXISTS role_assignments;

migrations/tenant/0002_role_assignments.up.sql

@@ -0,0 +1,23 @@
+-- Rollenzuweisung pro Benutzer (RBAC-01, siehe core-kanban/tickets/RBAC-01.md).
+-- Nur 'user' und 'tenant_admin' sind hier zuweisbar — 'superadmin' lebt
+-- mandantenuebergreifend in der Registry (IAM-01 SuperadminStore) und hat
+-- daher bewusst KEINE Zeile in dieser tenant-lokalen Tabelle (Akzeptanz-
+-- kriterium 1: nur Zuweisungen innerhalb der erlaubten Matrix).
+CREATE TABLE role_assignments (
+    user_id     UUID PRIMARY KEY REFERENCES users(id),
+    role        TEXT NOT NULL CHECK (role IN ('user', 'tenant_admin')),
+    granted_by  TEXT NOT NULL,
+    granted_at  TIMESTAMPTZ NOT NULL DEFAULT now()
+);
+
+-- Vollstaendige Historie jeder Rollenaenderung (Akzeptanzkriterium 3: wer
+-- hat wann welche Rolle vergeben).
+CREATE TABLE role_assignment_history (
+    id          UUID PRIMARY KEY DEFAULT gen_random_uuid(),
+    user_id     UUID NOT NULL,
+    role        TEXT NOT NULL,
+    granted_by  TEXT NOT NULL,
+    granted_at  TIMESTAMPTZ NOT NULL DEFAULT now()
+);
+
+CREATE INDEX role_assignment_history_user_idx ON role_assignment_history (user_id, granted_at);

migrations/tenant/0003_ldap.down.sql

@@ -0,0 +1,2 @@
+DROP TABLE IF EXISTS ldap_group_role_mapping;
+DROP TABLE IF EXISTS ldap_config;

migrations/tenant/0003_ldap.up.sql

@@ -0,0 +1,22 @@
+-- LDAP/Active-Directory-Anbindung (IAM-05, siehe core-kanban/tickets/IAM-05.md).
+-- Bind-Credentials liegen NIE in dieser Tabelle im Klartext — bind_password_env
+-- nennt nur den Namen einer Umgebungsvariable, aus der das Passwort zur
+-- Laufzeit gelesen wird (Projekt-Konvention: Secrets nur ueber Env-Vars).
+CREATE TABLE ldap_config (
+    id                 BOOLEAN PRIMARY KEY DEFAULT true CHECK (id),
+    url                TEXT NOT NULL,
+    bind_dn            TEXT NOT NULL,
+    bind_password_env  TEXT NOT NULL,
+    base_dn            TEXT NOT NULL,
+    user_filter        TEXT NOT NULL DEFAULT '(objectClass=person)',
+    updated_at         TIMESTAMPTZ NOT NULL DEFAULT now()
+);
+
+-- Erlaubnis-Matrix LDAP-Gruppe -> NEXARCH-Rolle (Akzeptanzkriterium 3).
+-- Eine LDAP-Gruppe OHNE Eintrag hier vergibt KEINE Rolle — kein impliziter
+-- Zugriff durch Gruppennamen-Zufall (Privilege-Escalation-Schutz, siehe
+-- "Bekannte Fehler vermeiden" im Ticket).
+CREATE TABLE ldap_group_role_mapping (
+    ldap_group TEXT PRIMARY KEY,
+    role       TEXT NOT NULL CHECK (role IN ('user', 'tenant_admin'))
+);

migrations/tenant/0003_totp.down.sql

@@ -1,3 +0,0 @@
-DROP TABLE IF EXISTS totp_policy;
-DROP TABLE IF EXISTS totp_recovery_codes;
-DROP TABLE IF EXISTS totp_credentials;

migrations/tenant/0003_totp.up.sql

@@ -1,23 +0,0 @@
--- Zwei-Faktor-Authentifizierung TOTP (IAM-04, siehe core-kanban/tickets/IAM-04.md).
-CREATE TABLE totp_credentials (
-    user_id    UUID PRIMARY KEY REFERENCES users(id),
-    secret     TEXT NOT NULL,
-    confirmed  BOOLEAN NOT NULL DEFAULT false,
-    created_at TIMESTAMPTZ NOT NULL DEFAULT now()
-);
-
--- code_hash enthaelt NIEMALS den Klartext-Wiederherstellungscode, nur dessen
--- SHA-256-Hash. used_at markiert einmalige Verwendbarkeit (Akzeptanzkriterium 3).
-CREATE TABLE totp_recovery_codes (
-    id         UUID PRIMARY KEY DEFAULT gen_random_uuid(),
-    user_id    UUID NOT NULL REFERENCES users(id),
-    code_hash  BYTEA NOT NULL UNIQUE,
-    used_at    TIMESTAMPTZ,
-    created_at TIMESTAMPTZ NOT NULL DEFAULT now()
-);
-
--- Erzwingbare Aktivierungspflicht pro Tenant (Singleton-Zeile, Modell C).
-CREATE TABLE totp_policy (
-    id       BOOLEAN PRIMARY KEY DEFAULT true CHECK (id),
-    required BOOLEAN NOT NULL DEFAULT false
-);