Alte Workspace-Adresse abgeschaltet statt weitergeleitet
Auf Filipes Wunsch: dogfather-universe.com/workspace/ soll nicht mehr existieren, er gibt den neuen Link selbst an Creator und Scouts weiter. Vorher stand dort eine Weiterleitung - die haette die alte Adresse auf unbestimmte Zeit am Leben gehalten. Der Server antwortet dort jetzt mit 410 Gone. Beides - 404 und 410 - heisst "gibt es hier nicht"; 410 sagt zusaetzlich "gab es, ist absichtlich weg, kommt nicht wieder". Genau der Sachverhalt. Suchmaschinen nehmen die Adresse dadurch schneller heraus, und in einem Protokoll ist ein 410 sofort als gewollt erkennbar, waehrend ein 404 immer nach Versehen aussieht. Statt einer nackten Fehlermeldung kommt eine kurze Hinweisseite mit der neuen Adresse. Wer hier landet, hat einen alten Link oder eine alte Verknuepfung und soll erfahren, wohin der Workspace gezogen ist, statt vor einem leeren Fenster zu sitzen. Bewusst OHNE Verweis zum Anklicken, damit niemand aus Versehen ein neues Lesezeichen auf den alten Weg setzt. Schnittstellen-Aufrufe unter /workspace/api/ bekommen JSON statt HTML - ein alter, noch offener Browser-Tab wuerde sonst die Hinweisseite als Daten zu lesen versuchen und einen unverstaendlichen Fehler zeigen. Die Pruefung deckt weiterhin 21 Faelle ab und ist mitgezogen: Der gefaehrlichste Fall heisst jetzt nicht mehr "Endlosschleife", sondern "die neue Adresse mit abschalten" - wer nicht auf den Hostnamen prueft, sperrt den Workspace fuer alle aus. Alle gruen, Gegenprobe schlaegt an. Diesmal vor dem Commit den Diff Datei fuer Datei kontrolliert - beim letzten Mal hatte ich hier eine fremde uncommittete Zeile mitgenommen und damit die Website fuer zwei Minuten abgeschossen. Co-Authored-By: Claude Opus 5 <[email protected]>
This commit is contained in:
@@ -1,13 +1,14 @@
|
||||
/* =====================================================================
|
||||
Prüft die Umzugs-Weiterleitung des Creator Workspace.
|
||||
Prüft, WELCHE Anfragen der abgeschalteten alten Workspace-Adresse
|
||||
zugeordnet werden.
|
||||
|
||||
Zwei Fehler waeren hier teuer, und beide sieht man einer Bedingung
|
||||
nicht an:
|
||||
|
||||
1. ENDLOSSCHLEIFE. Derselbe Dienst bedient die alte UND die neue
|
||||
Adresse. Wer nicht auf den Hostnamen prueft, leitet die neue
|
||||
Adresse immer wieder auf sich selbst -- der Workspace waere fuer
|
||||
alle unerreichbar, und zwar sofort nach dem Neustart.
|
||||
1. DIE NEUE ADRESSE MIT ABSCHALTEN. Derselbe Dienst bedient die alte
|
||||
UND die neue Adresse. Wer nicht auf den Hostnamen prueft, sperrt
|
||||
den Workspace fuer alle aus -- sofort nach dem Neustart und ohne
|
||||
dass es vorher irgendwo auffaellt.
|
||||
|
||||
2. PRAEFIX-IRRTUM. "faengt an mit /workspace" trifft auch
|
||||
/workspaceXYZ oder /workspace-alt -- Pfade, die nicht zum
|
||||
@@ -20,11 +21,11 @@
|
||||
|
||||
Aufruf: node server/pruef-workspace-umzug.mjs
|
||||
===================================================================== */
|
||||
import { sollUmgeleitetWerden, NEUE_ADRESSE } from "./workspace-umzug.js";
|
||||
import { istAlteWorkspaceAdresse, NEUE_ADRESSE } from "./workspace-umzug.js";
|
||||
|
||||
/* [Host, Pfad, erwartet, Begruendung] */
|
||||
const FAELLE = [
|
||||
// --- muss weiterleiten ---
|
||||
// --- muss abgeschaltet sein (410 Gone) ---
|
||||
["dogfather-universe.com", "/workspace/", true, "alte Adresse, Startseite"],
|
||||
["dogfather-universe.com", "/workspace", true, "alte Adresse, ohne Schraegstrich"],
|
||||
["dogfather-universe.com", "/workspace/start.html", true, "alte Adresse, Unterseite"],
|
||||
@@ -33,12 +34,12 @@ const FAELLE = [
|
||||
["DOGFATHER-UNIVERSE.COM", "/workspace/", true, "Grossschreibung darf egal sein"],
|
||||
["dogfather-universe.com:443", "/workspace/", true, "Portangabe darf egal sein"],
|
||||
|
||||
// --- darf NICHT weiterleiten: sonst Endlosschleife ---
|
||||
["workspace.dogfather-universe.com", "/workspace/", false, "NEUE Adresse -- sonst Endlosschleife"],
|
||||
// --- darf NIEMALS abgeschaltet werden: das ist der neue Ort ---
|
||||
["workspace.dogfather-universe.com", "/workspace/", false, "NEUE Adresse -- darf NIE abgeschaltet werden"],
|
||||
["workspace.dogfather-universe.com", "/workspace/start.html", false, "NEUE Adresse, Unterseite"],
|
||||
["workspace.dogfather-universe.com", "/workspace/api/aufgaben", false, "NEUE Adresse, Schnittstelle"],
|
||||
|
||||
// --- darf NICHT weiterleiten: gehoert nicht zum Workspace ---
|
||||
// --- darf nicht betroffen sein: gehoert nicht zum Workspace ---
|
||||
["dogfather-universe.com", "/", false, "Hauptseite"],
|
||||
["dogfather-universe.com", "/index.html", false, "Hauptseite"],
|
||||
["dogfather-universe.com", "/webdesign/", false, "anderer Bereich"],
|
||||
@@ -46,7 +47,7 @@ const FAELLE = [
|
||||
["dogfather-universe.com", "/workspace-alt/seite", false, "PRAEFIX-IRRTUM mit Bindestrich"],
|
||||
["dogfather-universe.com", "/assets/css/main.css", false, "gemeinsame Datei"],
|
||||
|
||||
// --- darf NICHT weiterleiten: lokale Tests wuerden sonst scheitern ---
|
||||
// --- darf nicht betroffen sein: lokale Tests wuerden sonst scheitern ---
|
||||
["127.0.0.1:4100", "/workspace/", false, "lokaler Testserver"],
|
||||
["localhost:4100", "/workspace/start.html", false, "lokaler Testserver"],
|
||||
|
||||
@@ -58,27 +59,27 @@ const FAELLE = [
|
||||
let geprueft = 0;
|
||||
const fehler = [];
|
||||
for (const [host, pfad, erwartet, grund] of FAELLE) {
|
||||
const ist = sollUmgeleitetWerden(host, pfad);
|
||||
const ist = istAlteWorkspaceAdresse(host, pfad);
|
||||
geprueft++;
|
||||
if (ist !== erwartet) {
|
||||
fehler.push(`${host || "(leer)"}${pfad} -> ${ist ? "leitet um" : "leitet NICHT um"}, erwartet ${erwartet ? "umleiten" : "nicht umleiten"} (${grund})`);
|
||||
fehler.push(`${host || "(leer)"}${pfad} -> ${ist ? "abgeschaltet" : "erreichbar"}, erwartet ${erwartet ? "abgeschaltet" : "erreichbar"} (${grund})`);
|
||||
}
|
||||
}
|
||||
|
||||
/* Gegenprobe: Kann diese Pruefung ueberhaupt "nein" sagen? Ein Fall, von
|
||||
dem wir wissen, dass er falsch ist, MUSS auffallen. */
|
||||
const gegenprobe = sollUmgeleitetWerden("dogfather-universe.com", "/workspace/") === true
|
||||
&& sollUmgeleitetWerden("workspace.dogfather-universe.com", "/workspace/") === false;
|
||||
const gegenprobe = istAlteWorkspaceAdresse("dogfather-universe.com", "/workspace/") === true
|
||||
&& istAlteWorkspaceAdresse("workspace.dogfather-universe.com", "/workspace/") === false;
|
||||
|
||||
/* Und das Ziel muss stimmen -- eine Weiterleitung auf die falsche Adresse
|
||||
waere schlimmer als gar keine. */
|
||||
/* Und die genannte neue Adresse muss stimmen -- sie steht auf der
|
||||
Hinweisseite, die abgemeldete Nutzer zu sehen bekommen. */
|
||||
const zielStimmt = NEUE_ADRESSE === "https://workspace.dogfather-universe.com";
|
||||
geprueft++;
|
||||
if (!zielStimmt) fehler.push(`Ziel ist "${NEUE_ADRESSE}", erwartet https://workspace.dogfather-universe.com`);
|
||||
|
||||
const SOLL = FAELLE.length + 1;
|
||||
console.log(`${geprueft} Faelle geprueft (erwartet ${SOLL}), ${fehler.length} Beanstandungen.`);
|
||||
console.log(`Gegenprobe (alt leitet um, neu nicht): ${gegenprobe ? "ja" : "NEIN"}`);
|
||||
console.log(`Gegenprobe (alt abgeschaltet, neu erreichbar): ${gegenprobe ? "ja" : "NEIN"}`);
|
||||
for (const f of fehler) console.log(" FEHL " + f);
|
||||
|
||||
if (geprueft !== SOLL) { console.log("FEHL: Anzahl stimmt nicht."); process.exit(1); }
|
||||
|
||||
Reference in New Issue
Block a user