Changeset e9128bc in Klonkt for src/routes/admin-migrate.js

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@…>

File:

• src/routes/admin-migrate.js (modified) (6 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,