Zehn Befunde, alle behoben. Die wichtigsten: scrypt lief mit N=2^15; das OWASP Password Storage Cheat Sheet nennt 2^17. Jetzt 2^17, alte Hashes bleiben gueltig und werden beim naechsten Anmelden still erneuert — das ist der einzige Moment, in dem das Klartextpasswort vorliegt. Passwoerter hatten keine Obergrenze. Ein Passwort von mehreren Megabyte haette scrypt beschaeftigt, und zwar an einem Endpunkt ohne Anmeldung. Jetzt 128 Zeichen, geprueft vor dem Hashen. Als CSRF-Schutz gab es nur SameSite=Lax. Das Merkblatt sagt ausdruecklich, dass das allein nicht genuegt — besonders beim Upload, der multipart entgegennimmt und damit von einem fremden Formular erreichbar waere. Jetzt wird bei jedem schreibenden Zugriff der Origin gegen den Host der Anfrage geprueft (nicht gegen BASE_URL, die Anwendung laeuft unter mehreren Domains). Es gab keinen einzigen Schutzheader — die Verwaltung liess sich einbetten und per Clickjacking fernsteuern. Jetzt frame-ancestors none, X-Frame-Options, nosniff, Referrer-Policy und Permissions-Policy; X-Powered-By ist weg. Sitzungen hatten keine absolute Obergrenze: eine gestohlene Sitzung haette durch regelmaessige Nutzung unbegrenzt weitergelebt. Jetzt SESSION_MAX_DAYS neben SESSION_IDLE_DAYS, beide einstellbar. Dazu: Secure auch hinter einem TLS-Proxy, weniger Auskunft ueber fremde Konten bei der Anmeldung, eine Bremse je Absender gegen breites Durchprobieren, Ersteinrichtung ohne Wettlauf, und abgelaufene Sitzungen werden aufgeraeumt. Die Datei heisst proxy.ts statt middleware.ts — Next 16 hat die Konvention umbenannt. /api/uploads bleibt davon ausgenommen: Der Proxy puffert Anfragekoerper nur bis 10 MB, eine Datei an der Groessengrenze kaeme sonst unvollstaendig an. Die Herkunftspruefung macht die Route dort selbst. Befund, Begruendungen und was bewusst offen bleibt: docs/sicherheit.md
112 lines
3.2 KiB
TypeScript
112 lines
3.2 KiB
TypeScript
import type { NextRequest } from "next/server";
|
|
import { z } from "zod";
|
|
|
|
import { getActor } from "@/lib/auth/actor";
|
|
import {
|
|
describePasswordRules,
|
|
hashPassword,
|
|
MAX_PASSWORD_LENGTH,
|
|
MIN_PASSWORD_LENGTH,
|
|
} from "@/lib/auth/password";
|
|
import { requirePermission } from "@/lib/auth/policy";
|
|
import { prisma } from "@/lib/db";
|
|
import { errorResponse, firstZodMessage, jsonError } from "@/lib/http";
|
|
import { QrRequestError } from "@/lib/qr/type-module";
|
|
|
|
export const dynamic = "force-dynamic";
|
|
|
|
const createSchema = z.object({
|
|
email: z
|
|
.string()
|
|
.trim()
|
|
.toLowerCase()
|
|
.min(1, { error: "Bitte gib eine E-Mail-Adresse ein." })
|
|
.max(254, { error: "Die Adresse ist zu lang." })
|
|
.refine((value) => /^[^\s@]+@[^\s@.]+\.[^\s@]+$/.test(value), {
|
|
error: "Das ist keine gültige E-Mail-Adresse.",
|
|
}),
|
|
name: z.string().trim().max(120).optional(),
|
|
password: z
|
|
.string()
|
|
.min(MIN_PASSWORD_LENGTH, { error: `Das Passwort ist zu kurz. ${describePasswordRules()}`, })
|
|
.max(MAX_PASSWORD_LENGTH, {
|
|
error: `Das Passwort darf höchstens ${MAX_PASSWORD_LENGTH} Zeichen lang sein.`,
|
|
}),
|
|
isInstanceAdmin: z.boolean().default(false),
|
|
});
|
|
|
|
export async function GET(): Promise<Response> {
|
|
try {
|
|
requirePermission(await getActor(), "instance:manage");
|
|
|
|
const users = await prisma.user.findMany({
|
|
orderBy: [{ isInstanceAdmin: "desc" }, { email: "asc" }],
|
|
include: {
|
|
memberships: { include: { group: { select: { id: true, name: true } } } },
|
|
_count: { select: { sessions: true } },
|
|
},
|
|
});
|
|
|
|
return Response.json({
|
|
users: users.map((user) => ({
|
|
id: user.id,
|
|
email: user.email,
|
|
name: user.name,
|
|
isInstanceAdmin: user.isInstanceAdmin,
|
|
isActive: user.isActive,
|
|
isLocked: Boolean(
|
|
user.lockedUntil && user.lockedUntil.getTime() > Date.now(),
|
|
),
|
|
lastLoginAt: user.lastLoginAt?.toISOString() ?? null,
|
|
createdAt: user.createdAt.toISOString(),
|
|
activeSessions: user._count.sessions,
|
|
memberships: user.memberships.map((m) => ({
|
|
groupId: m.group.id,
|
|
groupName: m.group.name,
|
|
role: m.role,
|
|
})),
|
|
})),
|
|
});
|
|
} catch (error) {
|
|
return errorResponse(error);
|
|
}
|
|
}
|
|
|
|
export async function POST(request: NextRequest): Promise<Response> {
|
|
try {
|
|
requirePermission(await getActor(), "instance:manage");
|
|
|
|
const parsed = createSchema.safeParse(await request.json());
|
|
if (!parsed.success) {
|
|
return jsonError(firstZodMessage(parsed.error));
|
|
}
|
|
|
|
const existing = await prisma.user.findUnique({
|
|
where: { email: parsed.data.email },
|
|
select: { id: true },
|
|
});
|
|
if (existing) {
|
|
throw new QrRequestError(
|
|
"Diese E-Mail-Adresse ist bereits vergeben.",
|
|
409,
|
|
);
|
|
}
|
|
|
|
const user = await prisma.user.create({
|
|
data: {
|
|
email: parsed.data.email,
|
|
name: parsed.data.name || null,
|
|
passwordHash: await hashPassword(parsed.data.password),
|
|
isInstanceAdmin: parsed.data.isInstanceAdmin,
|
|
},
|
|
});
|
|
|
|
return Response.json(
|
|
{ user: { id: user.id, email: user.email } },
|
|
{ status: 201 },
|
|
);
|
|
} catch (error) {
|
|
return errorResponse(error);
|
|
}
|
|
}
|