Changeset e9128bc in Klonkt for src/routes

Timestamp:

08/14/2026 02:12:31 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

fc664ce

Parents:

dc802b4

Message:

Alle migratie-opties bij elkaar op één pagina

Robin: ik wil alle migratie-opties op de migrate-pagina hebben, dus ook
fediverse alias en move.

Ze stonden op de site-bewerkpagina, tussen de kleuren, de feedinstellingen en
de custom head-html. Dat is de verkeerde plek in twee opzichten: het zijn stap
1 en de laatste stap van een verhuizing, en dat scherm gaat over hoe je site
eruitziet. Een verhuizing is geen uiterlijk.

De pagina vertelt nu het verhaal in de goede volgorde:

1 je oude account claimen (alsoKnownAs)

meenemen (zip)
ophalen bij je oude Klonkt
hierheen halen (zip)

4 de verhuizing aankondigen (Move)

Claimen bovenaan omdat de bron zonder die claim niets afgeeft en de Move
weigert met no_backreference. Aankondigen onderaan met een rode knop: het is de
enige stap op deze pagina die niet terug te draaien is.

De verhuisknop post naar de BESTAANDE /admin/sites/:slug/move. Een tweede
implementatie van een onomkeerbare actie is precies wat je niet wilt; die route
kreeg alleen een next mee zodat je terugkomt waar je vandaan kwam.

EEN VAL DIE IK ONDERWEG MAAKTE. De opslagroute van het site-bewerkscherm
schreef ap_aliases uit het formulier. Haal je dat veld weg, dan komt er niets
binnen, en dan wist een gewone opslag je claim op je oude account. Je verandert
je accentkleur en drie stappen later weigert de Move met no_backreference,
zonder dat iets die twee met elkaar verbindt. Nu geldt: veld niet meegestuurd
is niet aanraken.

Changed files:
src/routes/admin-migrate.js

• POST /admin/migrate/aliases, met de bestaande parseApAliases
• aliassen en moved_to in de locals van alle drie de renders

src/routes/admin-sites.js

• parseApAliases geexporteerd in plaats van gekopieerd
• de save raakt ap_aliases alleen aan als het veld er echt is
• /move mag terugsturen naar een eigen pad (geen open redirect)

src/views/pages/admin-migrate.ejs

• alias bovenaan, verhuizen onderaan met een gewaarschuwde rode knop

src/views/pages/admin-site-edit.ejs

• beide blokken eruit, een wegwijzer ervoor in de plaats

src/services/i18n.js

• mig.alias_* en mig.move_* plus asite.moved_to_migrate, in nl/en/de

New file:
test/migrate-page.test.js

• de velden staan op de goede pagina en niet meer op de oude
• de volgorde: claimen boven ophalen, aankondigen onderaan
• en een alias overleeft het opslaan van je uiterlijk

remarks: gedraaid op een echte instantie. Alias opslaan werkt, een onzin-adres
geeft een nette fout EN laat de bestaande alias staan, en de pagina rendert
schoon in nl/en/de. Suite 1005 groen.

-robo
Co-Authored-By: Claude Opus 4.8 <noreply@…>

Location:

src/routes

Files:

• admin-migrate.js (modified) (6 diffs)
• admin-sites.js (modified) (2 diffs)

src/routes/admin-migrate.js

@@ -35 +35 @@
 import { buildArchive, zipArchive } from '../services/ArchiveExportService.js';
 import { readArchiveZip, importArchive } from '../services/ArchiveImportService.js';
+import { parseApAliases } from './admin-sites.js';
+import db from '../config/database.js';
 
 const router = express.Router();
@@ -72 +74 @@
 }
 
+/** De aliassen als tekst, een per regel, zoals het invoerveld ze wil. */
+function aliasTekst(site) {
+  try { return (JSON.parse((site && site.ap_aliases) || '[]') || []).join('\n'); } catch { return ''; }
+}
+
 /**
  * Waar zouden we vandaan kunnen halen? De alias die we zelf claimen (FEP-7628
@@ -94 +101 @@
     telling, fout, mb,
     verslag: null,
-    bron: bronKandidaat(site),
+    bron: bronKandidaat(site), aliassen: aliasTekst(site), movedTo: (site && site.moved_to) || null,
     mig: site ? Migration.migrationStatus(site.slug) : null,
     haalVerslag: null,
@@ -133 +140 @@
     pageTitle: 'Migreren', bodyClass: 'on-special',
     telling, fout: null, mb, verslag: null,
-    bron: opgegeven || bronKandidaat(site),
+    bron: opgegeven || bronKandidaat(site), aliassen: aliasTekst(site), movedTo: site.moved_to || null,
     mig: Migration.migrationStatus(site.slug),
     haalVerslag: r,
@@ -139 +146 @@
     error: null,
   });
+});
+
+/**
+ * Je oude account claimen (FEP-7628 alsoKnownAs).
+ *
+ * Stond op de site-bewerkpagina, tussen de kleuren en de feedinstellingen. Maar
+ * dit is stap EEN van een verhuizing, en zonder deze claim weigert de oude
+ * instantie de Move met `no_backreference`. Het hoort dus hier, boven de knop
+ * die hem nodig heeft.
+ */
+router.post('/aliases', requireGod, async (req, res) => {
+  const site = res.locals.site;
+  if (!site) return res.redirect('/admin/migrate?error=' + encodeURIComponent('Geen site'));
+  const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
+  let lijst;
+  try {
+    lijst = await parseApAliases(req.body.ap_aliases, ActivityPubService.actorId(base, site.slug));
+  } catch (e) {
+    // Welke regel niet deugde, niet alleen DAT er iets niet deugde.
+    return res.redirect('/admin/migrate?error=' + encodeURIComponent(`Onbruikbaar adres: ${e && e.message}`));
+  }
+  db.prepare('UPDATE sites SET ap_aliases = ? WHERE slug = ?').run(JSON.stringify(lijst), site.slug);
+  res.redirect('/admin/migrate?success=' + encodeURIComponent(
+    lijst.length ? `${lijst.length} adres(sen) opgeslagen als jouw vorige account.` : 'Aliassen leeggemaakt.'));
 });
 
@@ -195 +226 @@
     pageTitle: 'Migreren', bodyClass: 'on-special',
     telling: tellen(site).telling, fout: null, mb,
-    bron: bronKandidaat(site), mig: Migration.migrationStatus(site.slug), haalVerslag: null,
+    bron: bronKandidaat(site), aliassen: aliasTekst(site), movedTo: site.moved_to || null,
+    mig: Migration.migrationStatus(site.slug), haalVerslag: null,
     verslag: verslag ? { ...verslag, echt, overschrijf, bestand: req.file.originalname, bytes: req.file.buffer.length } : null,
     success: (echt && verslag && !fout) ? 'Archief geïmporteerd' : null,

src/routes/admin-sites.js

@@ -80 +80 @@
  * Move fail at the old server with no hint why. Throws the offending line.
  */
-async function parseApAliases(raw, ownActorUri) {
+export async function parseApAliases(raw, ownActorUri) {
   const lines = String(raw || '').split(/\r?\n/).map((s) => s.trim()).filter(Boolean);
   if (lines.length > 5) throw new Error(lines[5] + ' (max 5)');
@@ -332 +332 @@
   // FEP-7628 aliases — validated/resolved before anything is written.
   const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
-  let apAliasesJson = null;
-  try {
-    const arr = await parseApAliases(f.ap_aliases, AP.actorId(base, req.params.slug));
-    apAliasesJson = arr.length ? JSON.stringify(arr) : null;
-  } catch (e) {
-    return res.redirect(`/admin/sites/${req.params.slug}/edit?error=` + encodeURIComponent(`Alias niet herkend of niet vindbaar: ${e.message}`));
+  // Het INVOERVELD staat hier sinds 14-8 niet meer: aliassen horen bij
+  // Migreren. Dit formulier mag ze dus niet aanraken, en al helemaal niet
+  // leegmaken omdat het veld ontbreekt. Anders verlies je je claim op je oude
+  // account door je kleuren aan te passen, en weigert de Move daarna met
+  // no_backreference. Alleen verwerken als het veld ECHT is meegestuurd, zodat
+  // een oude gecachte pagina die hem nog wel heeft blijft werken.
+  let apAliasesJson = site.ap_aliases || null;
+  if (Object.prototype.hasOwnProperty.call(f, 'ap_aliases')) {
+    try {
+      const arr = await parseApAliases(f.ap_aliases, AP.actorId(base, req.params.slug));
+      apAliasesJson = arr.length ? JSON.stringify(arr) : null;
+    } catch (e) {
+      return res.redirect(`/admin/sites/${req.params.slug}/edit?error=` + encodeURIComponent(`Alias niet herkend of niet vindbaar: ${e.message}`));
+    }
   }