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
@@ -0,0 +1,189 @@
|
||||
// @vitest-environment node
|
||||
import { afterAll, beforeAll, describe, expect, it } from 'vitest'
|
||||
import { createPrismaClient, loadEnvFromRepoRoot, type PrismaClient } from '@afis/db'
|
||||
import { visibleCaseWhere, visibleEntryWhere } from '@/lib/access/caseFilter'
|
||||
import type { EffectivePermissions, Permission } from '@/lib/auth/permissions'
|
||||
|
||||
/**
|
||||
* Die Negativtests aus docs/03-auth-und-rechte.md.
|
||||
*
|
||||
* Sie laufen gegen eine echte Datenbank, weil genau das der Punkt ist: die
|
||||
* Prüfung muss in der Abfrage stecken, nicht in einem Filter danach. Ein Test
|
||||
* gegen eine ausgedachte Liste würde die Aussage nicht treffen.
|
||||
*/
|
||||
loadEnvFromRepoRoot()
|
||||
const url = process.env.DIRECT_DATABASE_URL ?? process.env.DATABASE_URL
|
||||
const suite = url ? describe : describe.skip
|
||||
|
||||
let db: PrismaClient
|
||||
const marke = `zugriff-${Date.now()}`
|
||||
|
||||
function perms(
|
||||
userId: string,
|
||||
clearanceLevel: number,
|
||||
options: { isAdmin?: boolean; permissions?: Permission[] } = {},
|
||||
): EffectivePermissions {
|
||||
return {
|
||||
userId,
|
||||
clearanceLevel,
|
||||
permissions: new Set<Permission>(options.permissions ?? []),
|
||||
isAdmin: options.isAdmin ?? false,
|
||||
hasMembership: true,
|
||||
}
|
||||
}
|
||||
|
||||
suite('Sichtbarkeit von Akten', () => {
|
||||
const ids = {
|
||||
offen: `${marke}-offen`,
|
||||
geheim: `${marke}-geheim`,
|
||||
versiegelt: `${marke}-versiegelt`,
|
||||
}
|
||||
const nutzer = {
|
||||
stufe1: `${marke}-stufe1`,
|
||||
stufe2: `${marke}-stufe2`,
|
||||
stufe5: `${marke}-stufe5`,
|
||||
beteiligt: `${marke}-beteiligt`,
|
||||
admin: `${marke}-admin`,
|
||||
}
|
||||
|
||||
beforeAll(async () => {
|
||||
db = createPrismaClient(url)
|
||||
|
||||
const template = await db.caseTemplate.create({
|
||||
data: { key: `${marke}-vorlage`, version: 1, name: 'Prüfvorlage', fields: [] },
|
||||
})
|
||||
|
||||
for (const id of Object.values(nutzer)) {
|
||||
await db.user.create({ data: { id, username: id, displayName: id } })
|
||||
}
|
||||
|
||||
const gemeinsam = { templateId: template.id, templateKey: template.key, createdById: nutzer.stufe5 }
|
||||
|
||||
await db.case.create({
|
||||
data: {
|
||||
...gemeinsam,
|
||||
id: ids.offen,
|
||||
caseNumber: `AfiS-9001-${String(Date.now()).slice(-4)}`,
|
||||
title: 'Offener Vorgang',
|
||||
classification: 1,
|
||||
status: 'OPEN',
|
||||
},
|
||||
})
|
||||
|
||||
await db.case.create({
|
||||
data: {
|
||||
...gemeinsam,
|
||||
id: ids.geheim,
|
||||
caseNumber: `AfiS-9002-${String(Date.now()).slice(-4)}`,
|
||||
title: 'Geheimer Vorgang',
|
||||
classification: 4,
|
||||
status: 'OPEN',
|
||||
},
|
||||
})
|
||||
|
||||
await db.case.create({
|
||||
data: {
|
||||
...gemeinsam,
|
||||
id: ids.versiegelt,
|
||||
caseNumber: `AfiS-9003-${String(Date.now()).slice(-4)}`,
|
||||
title: 'Versiegelter Vorgang',
|
||||
classification: 1,
|
||||
status: 'SEALED',
|
||||
},
|
||||
})
|
||||
|
||||
// Beteiligter an der geheimen Akte, obwohl seine Stufe nicht reicht.
|
||||
await db.caseParticipant.create({
|
||||
data: { caseId: ids.geheim, userId: nutzer.beteiligt, role: 'WITNESS' },
|
||||
})
|
||||
|
||||
// Ausgeschlossen von der offenen Akte, obwohl Stufe 5 und Admin.
|
||||
await db.caseExclusion.createMany({
|
||||
data: [
|
||||
{ caseId: ids.offen, userId: nutzer.stufe5, reason: 'Betroffener' },
|
||||
{ caseId: ids.offen, userId: nutzer.admin, reason: 'Betroffener' },
|
||||
],
|
||||
})
|
||||
})
|
||||
|
||||
afterAll(async () => {
|
||||
await db.$disconnect()
|
||||
})
|
||||
|
||||
async function sichtbar(p: EffectivePermissions): Promise<string[]> {
|
||||
const rows = await db.case.findMany({
|
||||
where: { AND: [visibleCaseWhere(p), { id: { in: Object.values(ids) } }] },
|
||||
select: { id: true },
|
||||
})
|
||||
return rows.map((r) => r.id).sort()
|
||||
}
|
||||
|
||||
it('zeigt einer Stufe 2 die Stufe-4-Akte nicht, auch nicht als Treffer', async () => {
|
||||
const p = perms(nutzer.stufe2, 2)
|
||||
const treffer = await sichtbar(p)
|
||||
expect(treffer).not.toContain(ids.geheim)
|
||||
expect(treffer).toContain(ids.offen)
|
||||
|
||||
// Auch die Zählung darf sie nicht enthalten, sonst verrät die Lücke sie.
|
||||
const anzahl = await db.case.count({
|
||||
where: { AND: [visibleCaseWhere(p), { id: { in: Object.values(ids) } }] },
|
||||
})
|
||||
expect(anzahl).toBe(treffer.length)
|
||||
})
|
||||
|
||||
it('gibt einem ausgeschlossenen Nutzer mit Stufe 5 keinen Zugriff', async () => {
|
||||
const treffer = await sichtbar(perms(nutzer.stufe5, 5))
|
||||
expect(treffer).not.toContain(ids.offen)
|
||||
})
|
||||
|
||||
it('schlägt selbst den Admin, wenn er ausgeschlossen ist', async () => {
|
||||
// Der Ausschluss deckt den Fall ab, dass gegen einen hochrangigen
|
||||
// Mitarbeiter ermittelt wird. Der Administrator ist einer.
|
||||
const treffer = await sichtbar(perms(nutzer.admin, 5, { isAdmin: true }))
|
||||
expect(treffer).not.toContain(ids.offen)
|
||||
expect(treffer).toContain(ids.geheim)
|
||||
})
|
||||
|
||||
it('lässt einen Beteiligten auch über seiner Stufe lesen', async () => {
|
||||
const treffer = await sichtbar(perms(nutzer.beteiligt, 1))
|
||||
expect(treffer).toContain(ids.geheim)
|
||||
})
|
||||
|
||||
it('verbirgt versiegelte Akten unterhalb von Stufe 5', async () => {
|
||||
expect(await sichtbar(perms(nutzer.stufe2, 2))).not.toContain(ids.versiegelt)
|
||||
expect(await sichtbar(perms(nutzer.stufe1, 1))).not.toContain(ids.versiegelt)
|
||||
expect(await sichtbar(perms(nutzer.stufe5, 5))).toContain(ids.versiegelt)
|
||||
})
|
||||
|
||||
it('verbirgt Verlaufseinträge oberhalb der Stufe des Lesers', async () => {
|
||||
await db.caseEntry.createMany({
|
||||
data: [
|
||||
{
|
||||
id: `${marke}-eintrag-offen`,
|
||||
caseId: ids.offen,
|
||||
type: 'NOTE',
|
||||
body: 'Für alle lesbar',
|
||||
authorId: nutzer.stufe1,
|
||||
minClearance: 0,
|
||||
},
|
||||
{
|
||||
id: `${marke}-eintrag-hoch`,
|
||||
caseId: ids.offen,
|
||||
type: 'NOTE',
|
||||
body: 'Nur ab Stufe 4',
|
||||
authorId: nutzer.stufe1,
|
||||
minClearance: 4,
|
||||
},
|
||||
],
|
||||
})
|
||||
|
||||
const p = perms(nutzer.stufe1, 1)
|
||||
const eintraege = await db.caseEntry.findMany({
|
||||
where: { AND: [visibleEntryWhere(p), { caseId: ids.offen }] },
|
||||
select: { id: true },
|
||||
})
|
||||
const gefunden = eintraege.map((e) => e.id)
|
||||
expect(gefunden).toContain(`${marke}-eintrag-offen`)
|
||||
expect(gefunden).not.toContain(`${marke}-eintrag-hoch`)
|
||||
})
|
||||
})
|
||||
Reference in New Issue
Block a user