Sicherheit: SVG-Upload für Logo/Favicon nicht mehr erlaubt
Der Branding-Upload akzeptierte bisher jeden Dateityp, der clientseitig mit "image/" beginnt, inkl. SVG. Anders als beim vorherigen Supabase- Storage-Bucket (andere Origin) wird die Datei jetzt auf der eigenen Origin unter /branding/[filename] ausgeliefert – eine SVG mit eingebettetem <script> hätte bei direktem Aufruf der Datei-URL (z. B. "Bild in neuem Tab öffnen") mit dem Session-Cookie eines anderen Admins ausgeführt werden können und wäre dabei auch am CSRF-Schutz vorbeigekommen (Origin stimmt ja). Upload-Route erlaubt jetzt nur noch eine Whitelist an Rasterformaten (png/jpg/webp/gif/ico), die Dateiendung wird aus dem geprüften MIME-Typ abgeleitet statt aus dem vom Client gesendeten Dateinamen. Die Ausliefer-Route kennt "svg" als Content-Type nicht mehr (Defense in Depth). Gefunden bei der Sicherheitsprüfung vor dem Merge nach master.
This commit is contained in:
@@ -1,11 +1,15 @@
|
||||
import { readBrandingFile } from "@/lib/branding";
|
||||
|
||||
// Kein svg: würde mit Content-Type image/svg+xml ausgeliefert und ist
|
||||
// bei direkter Navigation auf die Datei aktiv ausführbar (eingebettetes
|
||||
// <script> läuft dann same-origin inkl. Session-Cookie) – anders als bei
|
||||
// Einbettung per <img>/<link rel="icon">. Nur ungefährliche Rasterformate
|
||||
// zulassen, siehe auch die Whitelist beim Upload in api/admin/branding.
|
||||
const CONTENT_TYPES: Record<string, string> = {
|
||||
png: "image/png",
|
||||
jpg: "image/jpeg",
|
||||
jpeg: "image/jpeg",
|
||||
webp: "image/webp",
|
||||
svg: "image/svg+xml",
|
||||
ico: "image/x-icon",
|
||||
gif: "image/gif",
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user