Vorlagen-JSON abweisen statt still verwerfen, Platzhalter ohne Prototypkette

Durchgesehen, was mit Text passiert, der nicht als Text ankommt. Die Grundlage
war schon in Ordnung: genau ein dangerouslySetInnerHTML mit einer Konstanten,
kein eval, kein $queryRawUnsafe, keine Weiterleitung aus einem Adressparameter,
Dateien mit nosniff und sandbox-CSP als Anhang statt inline.

Drei Ränder waren offen.

**Vorlagen.** Das zod-Schema hatte einen festen Satz Feldarten und speicherte
das Geprüfte, verwarf unbekannte Schlüssel aber stillschweigend. Mit
strictObject fällt ein `typ` statt `type` auf, statt eine Vorlage zu
hinterlassen, die nicht tut, was dasteht.

**Platzhalter lasen über die Prototypkette.** Beim Durchprobieren gefunden:
`constructor` ist ein gültiger Feldschlüssel, die Schlüsselregel erlaubt
Kleinbuchstaben. Ein Baustein mit {{constructor}} bekam den Quelltext der
geerbten Funktion in den Text geschrieben — kein ausgeführter Code, aber
Innenleben in einem Dokument, das weitergereicht wird. Jetzt Object.hasOwn.

**Farbmarken aus der Datenbank** landen als Name einer CSS-Variablen in einem
style-Attribut. Heute setzt das nur der Seed, und eine zweite Deklaration liesse
sich darüber ohnehin nicht einschleusen — aber der Kommentar sagt „im
Adminbereich pflegbar", und was aus der Datenbank in ein Stilattribut geht,
wird geprüft.

Dreizehn Prüfungen, die es mit echten Versuchen probieren. Bösartiger Text
bleibt dabei ausdrücklich erlaubt: eine Feldbeschriftung darf
<script>alert(1)</script> heissen und sieht dann genau so aus. Verboten ist,
dass daraus ein Element wird.

Nebenbei: eine Vorlagenänderung stand im Zugriffsprotokoll als roles.update.

web/lib/templates/fields.ts

@@ -91,7 +91,13 @@ export function fillPlaceholders(
   fallback = (key: string) => `[${key}]`,
 ): string {
   return text.replace(PLACEHOLDER, (_match, name: string) => {
-    const value = values[name]
+    // **Nur eigene Eigenschaften.** `values[name]` lief über die
+    // Prototypkette: ein Baustein mit dem Platzhalter `{{constructor}}` oder
+    // `{{toString}}` bekam den Quelltext der geerbten Funktion in den Text
+    // geschrieben — die Feldschlüssel sind Kleinbuchstaben, und `constructor`
+    // ist einer. Kein Code, der ausgeführt wird, aber Innenleben, das in einem
+    // Dokument nichts zu suchen hat.
+    const value = Object.hasOwn(values, name) ? values[name] : undefined
     if (value === undefined || value === null || value === '') return fallback(name)
     return Array.isArray(value) ? value.join(', ') : String(value)
   })

web/lib/templates/schema.ts

@@ -22,7 +22,14 @@ const gemeinsam = {
   minClearance: z.number().int().min(0).max(5).optional(),
 }
 
-const inputField = z.object({
+/**
+ * `.strict()` und nicht die Voreinstellung: zod wuerde unbekannte Schluessel
+ * **stillschweigend verwerfen**. Fuer eine Vorlage, die von Hand als JSON
+ * eingegeben wird, ist das die schlechtere Antwort — wer `typ` statt `type`
+ * schreibt, bekaeme eine gespeicherte Vorlage, die nicht tut, was dasteht.
+ * Und was nicht ankommt, kann auch nichts einschleusen.
+ */
+const inputField = z.strictObject({
   ...gemeinsam,
   type: z.enum(INPUT_TYPES),
   required: z.boolean().optional(),
@@ -36,7 +43,7 @@ const inputField = z.object({
   dateKey: key.optional(),
 })
 
-const staticBlock = z.object({
+const staticBlock = z.strictObject({
   ...gemeinsam,
   type: z.enum(STATIC_TYPES),
   text: z.string().max(20_000).optional(),