Phase 1: Anmeldung, Rechte und Rollenverwaltung

Auth.js übernimmt nur den OAuth-Ablauf mit state und PKCE, Scope identify. Das
Zugangstoken des Nutzers wird danach verworfen: die Rollen holt der Bot mit
seinem eigenen Token. Die Sitzung steht in der Datenbank, im Cookie liegt nur
userId und sessionId. Ohne die Zeile in Session ist das Cookie wertlos, und nur
so lässt sich eine Sitzung zurückziehen.

Wer sich anmelden darf, steckt in lib/auth/signInPolicy.ts statt im Callback:
das ist die Regel, an der alles hängt, und sie gehört einzeln prüfbar. Kein
Guild-Mitglied, kein Zugang. Keine Rolle mit grantsMembership, kein Zugang.
Notfall-Admins umgehen die Rolle, aber nicht die Guild-Mitgliedschaft.

RoleMapping trägt jetzt grantsMembership, category und isPresent. Der Worker
spiegelt jede Rolle der Guild hierher, damit im Adminbereich nichts von Hand
eingetragen werden muss. Gelöschte Rollen werden nicht entfernt, sondern auf
isPresent = false gesetzt: beim Umzug auf den echten Server bleiben sie als
Vorlage stehen.

Der Worker gleicht über Gateway-Ereignisse und zusätzlich alle fünf Minuten
voll ab. Verliert jemand seine AfiS-Rolle, beendet er dessen Sitzungen sofort
und setzt isActive auf false — der Datensatz bleibt, damit Autorenangaben in
alten Akten gültig bleiben. Das passiert im Worker und nicht in der Web-App,
weil es dort erst beim nächsten Request des Betroffenen griffe.

Zwei Widersprüche der Dokumente aufgelöst, festgehalten in
docs/13-entscheidungen.md: der Ausschluss schlägt auch den Admin (das
Codebeispiel in 03-auth-und-rechte.md übergeht ihn, die Reihenfolge eine Zeile
darüber nicht), und versiegelte Akten fallen im Where-Fragment heraus statt in
einer Prüfung danach.

45 Tests, davon die Negativfälle aus docs/03 gegen eine echte Datenbank: Stufe 2
sieht die Stufe-4-Akte nicht und findet sie auch nicht in der Trefferzahl, ein
Ausgeschlossener mit Stufe 5 kommt nicht heran, ein Rollenentzug beendet die
Sitzungen, und ein Ausfall der Discord-API sieht nicht aus wie eine verweigerte
Mitgliedschaft.
This commit is contained in:
Harlekin
2026-09-01 19:11:53 +02:00
parent c6401aeb4d
commit cb66839a05
36 changed files with 2339 additions and 98 deletions
+96
View File
@@ -0,0 +1,96 @@
import { afterAll, beforeAll, describe, expect, it } from 'vitest'
import { createPrismaClient, type PrismaClient } from '@afis/db'
import { enforceMembership, handleDeparture } from '../src/sync.js'
/**
* „Rollenentzug in Discord beendet laufende Sitzungen" aus
* docs/03-auth-und-rechte.md.
*
* Das Zeitbudget von fünf Sekunden ist im Betrieb die Summe aus
* Gateway-Ereignis und diesem Schritt. Geprüft wird hier der Schritt, der in
* unserer Hand liegt: die Sitzungen müssen weg sein, und der Datensatz muss
* stehen bleiben, damit Autorenangaben in alten Akten gültig bleiben.
*/
const url = process.env.DIRECT_DATABASE_URL ?? process.env.DATABASE_URL
const suite = url ? describe : describe.skip
let db: PrismaClient
const marke = `worker-${Date.now()}`
const rolleAfis = `${marke}-afis`
const rolleGast = `${marke}-gast`
suite('Sitzungen beim Rollenentzug', () => {
beforeAll(async () => {
db = createPrismaClient(url)
await db.roleMapping.createMany({
data: [
{ discordRoleId: rolleAfis, label: 'AfiS', category: 'MEMBERSHIP', grantsMembership: true },
{ discordRoleId: rolleGast, label: 'Gast', category: 'UNSORTED' },
],
})
})
afterAll(async () => {
await db.$disconnect()
})
async function nutzerMitSitzung(id: string, rollen: string[]): Promise<void> {
await db.user.create({ data: { id, username: id, displayName: id } })
if (rollen.length > 0) {
await db.userRole.createMany({
data: rollen.map((discordRoleId) => ({ userId: id, discordRoleId })),
})
}
await db.session.create({
data: { userId: id, expiresAt: new Date(Date.now() + 86_400_000) },
})
}
it('lässt eine Sitzung stehen, solange die AfiS-Rolle vorliegt', async () => {
const id = `${marke}-bleibt`
await nutzerMitSitzung(id, [rolleAfis])
const beendet = await enforceMembership(db, id)
expect(beendet).toBe(false)
expect(await db.session.count({ where: { userId: id } })).toBe(1)
})
it('beendet alle Sitzungen, sobald die AfiS-Rolle fehlt', async () => {
const id = `${marke}-verliert`
await nutzerMitSitzung(id, [rolleAfis, rolleGast])
await db.session.create({
data: { userId: id, expiresAt: new Date(Date.now() + 86_400_000) },
})
expect(await db.session.count({ where: { userId: id } })).toBe(2)
// Rollenentzug in Discord, hier direkt nachgestellt.
await db.userRole.deleteMany({ where: { userId: id, discordRoleId: rolleAfis } })
const start = Date.now()
const beendet = await enforceMembership(db, id)
const dauer = Date.now() - start
expect(beendet).toBe(true)
expect(await db.session.count({ where: { userId: id } })).toBe(0)
expect(dauer).toBeLessThan(5000)
// Der Datensatz bleibt, nur inaktiv.
const nutzer = await db.user.findUnique({ where: { id } })
expect(nutzer).not.toBeNull()
expect(nutzer?.isActive).toBe(false)
})
it('beendet Sitzungen beim Serveraustritt und behält den Nutzer', async () => {
const id = `${marke}-austritt`
await nutzerMitSitzung(id, [rolleAfis])
await handleDeparture(db, id)
expect(await db.session.count({ where: { userId: id } })).toBe(0)
expect(await db.userRole.count({ where: { userId: id } })).toBe(0)
const nutzer = await db.user.findUnique({ where: { id } })
expect(nutzer?.isActive).toBe(false)
})
})
+5
View File
@@ -0,0 +1,5 @@
import { loadEnvFromRepoRoot } from '@afis/db'
// Damit die Tests die lokale Entwicklungsdatenbank finden. In der CI stehen
// die Werte bereits in der Umgebung.
loadEnvFromRepoRoot()