Files
DogFatherGitandClaude Opus 5 6e46a08543 Portnummern werden abgeleitet, nicht mehr vergeben
Gemessen: 165 Pruefdateien, 129 verschiedene Nummern -- NEUNZEHN
doppelt, vier davon dreifach. Niemand hatte das gewollt; jede neue
Pruefung wurde von einer vorhandenen abgeschrieben, und die Nummer kam
mit. Meine eigene Notiz sagte "sieben" -- auch eine Bestandsliste
altert.

Der Waechter faengt den Schaden ab, aber er kann nur melden, was schon
passiert ist: Zwei Pruefungen mit derselben Nummer koennen nie
gleichzeitig laufen, und ein liegengebliebener Prozess der einen laesst
die andere abbrechen mit einer Meldung, die wie ein Befund aussieht.
Genau das ist mir am 20.09. zweimal passiert.

Jetzt leitet jede Datei ihre Nummer aus ihrer STELLE IM ALPHABET ab
(eigenerPort in helfer-port.mjs), zwei je Datei. Nicht ueber eine
Pruefsumme: Bei 165 Namen in 4900 Nummern waeren nach dem
Geburtstagsproblem rund DREI Zusammenstoesse zu erwarten -- ein Hash
tauscht eine sichtbare Doppelung gegen eine unsichtbare. Die Stelle im
Alphabet ist eindeutig von der Bauart her.

--- ZWEI FEHLER AUF DEM WEG, BEIDE LEHRREICH -------------------------

1. DER ERSTE VERSUCH WAR GRUEN UND KAPUTT. Ersetzt wurde mit einem
   Muster: "([^"]*4231[^"]*)". Das hielt

     { host: "127.0.0.1", port: 4231, path: "/404.html" }

   fuer eine Zeichenkette -- ein Muster kann eine oeffnende nicht von
   einer schliessenden Anfuehrung unterscheiden. Heraus kam

     { host: "127.0.0.1`, port: ${PORT}, path: `/404.html" }

   also GUELTIGER Code ohne port-Feld. `node --check` sagte gruen fuer
   alle 164 Dateien. Aufgefallen ist es erst, weil ich vier Vertreter
   gegen eine vorher gemessene Grundlinie laufen liess: pruef-schranke
   39/0 vorher, 38/1 nachher.

   Alles zurueckgenommen und mit einem Zerleger neu gemacht, der weiss,
   ob eine Stelle Code, Zeichenkette, Vorlage, Kommentar oder
   regulaerer Ausdruck ist. In pruef-ics stand die Nummer in einem
   regulaeren Ausdruck -- der wird jetzt gebaut statt hingeschrieben.

2. EIN MODUL, DAS BEIM IMPORTIEREN ARBEITET, IST EINE FALLE. Der
   zweite Durchgang importierte den ersten, um seine Mechanik zu
   benutzen -- und fuehrte dessen Hauptlauf gleich mit aus. Die
   zweiten Nummern wurden dadurch als erste behandelt, zwei Aufrufe
   bekamen dieselbe Nummer, und in pruef-content stand `const PORT`
   zweimal.

--- WAS DAS DAUERHAFT HAELT -----------------------------------------

pruef-portnummern.mjs (neu, 8 Pruefungen) fragt nicht "welche Nummern
sind doppelt", sondern "wer traegt ueberhaupt noch eine von Hand ein"
und "wer startet einen Server, ohne seine Nummer abzuleiten". Die
zweite Frage hat sofort etwas gefunden, das in KEINER Doppelungsliste
stand: pruef-push-weg belegte 4341 und 4342, rief den Waechter aber
gar nicht auf -- dieselbe Nummer wie pruef-agentur. Eine Liste zeigt
nur, was auf ihr steht.

Mit Gegenprobe: Eine unbekannte Datei bekommt keine geratene Nummer,
sondern einen Abbruch, und eine dritte Nummer je Datei gibt es nicht.

--- NACHGEMESSEN ----------------------------------------------------

Sechzehn Pruefungen gegen ihre vorher gemessene Grundlinie, je eine
Vertreterin jeder umgestellten Bauweise (eine Nummer, zwei Nummern,
Nummer in einer Zeichenkette, in einer Vorlage, in einem regulaeren
Ausdruck, dynamische Einfuhr, Nachtlauf mit zwei Laeufen):

  crew-adresse 132/0 · schranke 39/0 · content 45/0 · entwicklung 46/0
  anruf 127/0 · ics 37/0 · push-weg 20/0 · arten 28/0 · video 67/0
  kanalzeile 14/0 · agentur 62/0 · spicy 83/0 · creator-anlegen 50/0
  push-ziel 10/0 · portnummern 8/0

Alle exakt wie vorher. Zwei waren schon vorher rot und sind es
unveraendert geblieben (crew-wand-bild 41/4, teilen 14/3) -- per
`git stash` belegt, nicht angenommen.

Co-Authored-By: Claude Opus 5 <[email protected]>
2026-09-20 22:51:21 +02:00

177 lines
8.7 KiB
JavaScript

/* ===================================================================
Pruefung: DogFather und Manager koennen als zustaendige Person
eingetragen werden -- genau wie ein Scout.
Wunsch Filipe, 31.08.2026: "dogfather soll auch zaehlen wie
bananastift und patrick."
Der springende Punkt ist der letzte Abschnitt: Ein Eintrag auf
DogFather darf an den RECHTEN nichts aendern. Nur beim Scout
entscheidet die Zustaendigkeit zusaetzlich ueber die Sichtbarkeit.
=================================================================== */
import { mkdtempSync, rmSync } from "node:fs";
import { tmpdir } from "node:os";
import { join } from "node:path";
import { eigenerPort } from "./helfer-port.mjs";
const ordner = mkdtempSync(join(tmpdir(), "ws-betreuung-"));
process.env.WORKSPACE_DB = join(ordner, "workspace.db");
/* DER PORT MUSS MEINER SEIN (nachgeruestet 19.09.2026).
Ohne diese Zeile startet der eigene Server bei belegtem Port
STILL nicht -- und alles Folgende misst gegen einen fremden
Stand. Das ist der dritte Ausgang: belegt ist etwas anderes
als kaputt. */
const PORT = await eigenerPort(import.meta, "pruef-betreuung");
process.env.PORT = `${PORT}`;
process.env.SITE_ACCESS_SECRET = "lokaler-test";
process.env.SITE_ACCESS_CODE_DOGI = "DOGI-TEST-1";
process.env.SITE_ACCESS_CODE_VANVAN = "VANVAN-TEST-1";
process.env.SITE_PUBLIC_LAUNCH_AT = "2020-01-01T00:00:00+01:00";
const express = (await import("express")).default;
const ec = express.response.cookie;
express.response.cookie = function (n, w, o) { return ec.call(this, n, w, { ...(o || {}), secure: false }); };
await import("./index.js");
await new Promise((r) => setTimeout(r, 700));
const BASIS = `http://127.0.0.1:${PORT}`;
setTimeout(() => { console.log("ABBRUCH"); process.exit(1); }, 200_000).unref?.();
let fehler = 0;
const ok = (b, t) => { console.log((b ? " ok " : " FEHL ") + t); if (!b) fehler++; };
await fetch(BASIS + "/workspace/api/anmelden", {
method: "POST", headers: { "Content-Type": "application/json" },
body: JSON.stringify({ rolle: "admin", code: "x" }) }).catch(() => {});
const { DatabaseSync } = await import("node:sqlite");
const { scryptSync, randomBytes } = await import("node:crypto");
const d = new DatabaseSync(process.env.WORKSPACE_DB);
const jetzt = new Date().toISOString();
function anlegen(name, rolle, code) {
const salt = randomBytes(16).toString("hex");
const hash = scryptSync(code, salt, 64, { N: 32768, r: 8, p: 1, maxmem: 96 * 1024 * 1024 }).toString("hex");
d.prepare("INSERT INTO personen (name, rolle, code_hash, code_salt, code_n, aktiv, erstellt) VALUES (?,?,?,?,?,1,?)")
.run(name, rolle, hash, salt, 32768, jetzt);
return d.prepare("SELECT last_insert_rowid() AS id").get().id;
}
const idDogi = anlegen("Dogfather", "admin", "CODE-DOGI-0001");
const idManager = anlegen("Ein Manager", "manager", "CODE-MANA-0001");
const idBanana = anlegen("BananaStift", "scout", "CODE-BANA-0001");
const idPatrick = anlegen("Patrick", "scout", "CODE-PATR-0001");
const idTili = anlegen("Tili", "creator", "CODE-TILI-0001");
const idSpong = anlegen("SpongBob", "creator", "CODE-SPON-0001");
d.close();
const anmelden = async (rolle, code) => {
const a = await fetch(BASIS + "/workspace/api/anmelden", {
method: "POST", headers: { "Content-Type": "application/json" },
body: JSON.stringify({ rolle, code }), redirect: "manual" });
return (a.headers.getSetCookie?.() || []).map((c) => c.split(";")[0]).join("; ");
};
const ruf = (pfad, keks, opt = {}) => fetch(BASIS + pfad, {
...opt, headers: { cookie: keks, "Content-Type": "application/json", ...(opt.headers || {}) } });
const jsn = async (a) => { try { return await a.json(); } catch { return {}; } };
const setzen = (creator, betreuer, keks) => ruf(`/workspace/api/verwaltung/betreuung/${creator}`, keks, {
method: "PUT", body: JSON.stringify({ betreuer_id: betreuer }) });
const kDogi = await anmelden("admin", "CODE-DOGI-0001");
const kBanana = await anmelden("scout", "CODE-BANA-0001");
const kPatrick = await anmelden("scout", "CODE-PATR-0001");
console.log("\n1) Die Auswahl enthält jetzt alle betreuenden Rollen");
{
const { betreuer } = await jsn(await ruf("/workspace/api/verwaltung/personen", kDogi));
console.log(" ", betreuer.map((b) => `${b.name} (${b.rolle})`).join(" · "));
ok(betreuer.length === 4, `vier Auswahlmöglichkeiten (${betreuer.length})`);
ok(betreuer.map((b) => b.rolle).join(",") === "admin,manager,scout,scout",
`in der üblichen Reihenfolge (${betreuer.map((b) => b.rolle).join(", ")})`);
ok(betreuer[0].name === "Dogfather", `DogFather steht zuerst (${betreuer[0].name})`);
ok(!betreuer.some((b) => b.rolle === "creator"), "kein Creator in der Auswahl");
}
console.log("\n2) DogFather lässt sich eintragen wie ein Scout");
{
const a = await setzen(idTili, idDogi, kDogi);
const j = await jsn(a);
ok(a.ok, `DogFather als zuständig eingetragen (${a.status})`);
ok(j.betreuer_name === "Dogfather", `mit Namen zurückgemeldet (${j.betreuer_name})`);
const m = await setzen(idSpong, idManager, kDogi);
ok(m.ok, `und ein Manager genauso (${m.status})`);
const { personen } = await jsn(await ruf("/workspace/api/verwaltung/personen", kDogi));
const tili = personen.find((p) => p.name === "Tili");
ok(tili.betreuer_name === "Dogfather", `steht an der Person (${tili.betreuer_name})`);
/* Und DogFather zaehlt jetzt als Betreuer -- genau wie ein Scout. */
const dogi = personen.find((p) => p.name === "Dogfather");
ok(dogi.betreut_anzahl === 1, `DogFather betreut 1 Creator (${dogi.betreut_anzahl})`);
}
console.log("\n3) Der Hinweis über fehlende Zuständigkeit stimmt jetzt mit der Anzeige überein");
{
/* Das war der eigentliche Fehler: In der Auswahl stand "DogFather",
der Hinweis zaehlte dieselben Creator trotzdem als unbetreut. Zwei
Aussagen, ein Bildschirm. */
const { hinweise } = await jsn(await ruf("/workspace/api/hinweise", kDogi));
const ohne = hinweise.find((h) => h.art === "ohne_betreuung");
ok(!ohne, `kein Creator mehr ohne zuständige Person (${ohne ? ohne.anzahl : 0})`);
/* Gegenprobe: Zuruecksetzen auf "niemand" -- dann MUSS der Hinweis
wiederkommen, sonst waere er wertlos. */
await setzen(idTili, null, kDogi);
const zweite = await jsn(await ruf("/workspace/api/hinweise", kDogi));
const wieder = zweite.hinweise.find((h) => h.art === "ohne_betreuung");
ok(wieder?.anzahl === 1, `nach dem Zurücksetzen wieder gemeldet (${wieder?.anzahl})`);
await setzen(idTili, idDogi, kDogi);
}
console.log("\n4) Ein Creator kann nicht zuständig sein");
{
const a = await setzen(idTili, idSpong, kDogi);
ok(a.status === 400, `abgelehnt (${a.status})`);
console.log(" ->", (await jsn(a)).fehler);
}
console.log("\n5) DAS WICHTIGSTE: an den Rechten ändert das nichts");
{
/* Tili ist auf DogFather eingetragen. Kein Scout darf sie dadurch
sehen -- die Zustaendigkeit steuert die Sichtbarkeit NUR beim
Scout. Waere das anders, haette eine Anzeigeeinstellung still
Rechte vergeben. */
const b = await jsn(await ruf("/workspace/api/uebersicht/creator", kBanana));
ok((b.creator || []).length === 0,
`BananaStift sieht weiterhin keinen Creator (${(b.creator || []).length})`);
/* Und der Gegenbeweis: Auf einen Scout eingetragen, sieht genau DER
sie -- und nur er. */
await setzen(idTili, idPatrick, kDogi);
const p = await jsn(await ruf("/workspace/api/uebersicht/creator", kPatrick));
ok((p.creator || []).length === 1 && p.creator[0].name === "Tili",
`Patrick sieht Tili, sobald sie ihm zugeteilt ist (${(p.creator || []).map((c) => c.name).join(", ")})`);
const b2 = await jsn(await ruf("/workspace/api/uebersicht/creator", kBanana));
ok((b2.creator || []).length === 0,
`BananaStift immer noch nicht (${(b2.creator || []).length})`);
/* Zurueck auf DogFather -- Patrick verliert die Sicht wieder. */
await setzen(idTili, idDogi, kDogi);
const p2 = await jsn(await ruf("/workspace/api/uebersicht/creator", kPatrick));
ok((p2.creator || []).length === 0,
`und verliert sie wieder, sobald DogFather übernimmt (${(p2.creator || []).length})`);
/* Die Leitung sah ohnehin immer alle -- das darf sich nicht geaendert
haben, in keine Richtung. */
const dg = await jsn(await ruf("/workspace/api/uebersicht/creator", kDogi));
ok((dg.creator || []).length === 2, `DogFather sieht unverändert beide (${(dg.creator || []).length})`);
}
console.log("\n6) Ein Scout kann die Zuständigkeit nicht selbst setzen");
{
const a = await setzen(idSpong, idBanana, kBanana);
ok(a.status === 404, `Scout bekommt 404 (${a.status})`);
}
console.log(`\n${fehler === 0 ? "Alles in Ordnung." : fehler + " Prüfung(en) fehlgeschlagen."}`);
try { rmSync(ordner, { recursive: true, force: true }); } catch { /* egal */ }
process.exit(fehler === 0 ? 0 : 1);