Changeset f926d13 in Klonkt for src/routes/posts.js

Timestamp:

08/13/2026 07:37:56 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

47db4b5

Parents:

822d16d

Message:

De volglijst als bestand uploaden, niet alleen plakken

Robins vraag bij de echte verhuizing: hij downloadt een CSV en moet hem dan met de
hand openen en de inhoud overplakken. Dat is een omweg die niemand hoort te lopen.

Nu een bestandskiezer naast het plakveld. Multer met memoryStorage, want dit is een
lijstje adressen van een paar kilobyte dat na het lezen niets te zoeken heeft op de
server; op schijf schrijven zou alleen rommel achterlaten. Een gekozen bestand wint
van het plakveld: wie een bestand aanwijst bedoelt dat.

De BOM gaat eraf. Excel zet die voor een CSV, en zonder dat eraf te halen plakt hij
aan het eerste adres vast en is de eerste regel stil onbruikbaar.

En twee redirects in deze route wezen nog naar /following. Dat is de oude pagina
waar niets meer naartoe linkt (zie 822d16d), dus je kwam na een import op een
scherm waar je zelf niet meer weg kon. Beide naar /connect.

Changed files:
src/routes/posts.js

• multer memoryStorage, single('csvfile'), 512kb, 1 bestand
• bestand wint van plakveld, BOM eraf
• drie redirects van /following naar /connect

src/views/pages/connect.ejs

• bestandskiezer met accept=".csv,text/csv,text/plain", formulier op multipart

src/services/i18n.js

• tl.move_import_file in nl, en en de; het plakveld-label ingekort tot "Of plak de lijst hier"

test/following-csv.test.js

• test 14: een echt multipart-verzoek met een BOM ervoor, plus de controle dat het plakveld ernaast blijft werken

remarks: 927 groen in UTC en Europe/Amsterdam. De uploadweg is met een echte
multipart-POST tegen een draaiende express+multer getest, niet alleen beredeneerd.
De pagina apart gerenderd: bestandskiezer en multipart-enctype aanwezig, geen
onvertaalde sleutels.

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

File:

• src/routes/posts.js (modified) (3 diffs)

src/routes/posts.js

@@ -1294 +1294 @@
   const { followingCsv } = await import('../services/ArchiveExportService.js');
   const csv = site ? followingCsv(site.slug) : null;
-  if (!csv) return res.redirect('/following?error=' + encodeURIComponent('Je volgt nog niemand'));
+  if (!csv) return res.redirect('/connect?error=' + encodeURIComponent('Je volgt nog niemand'));
   res.set('Content-Type', 'text/csv; charset=utf-8');
   res.set('Content-Disposition', `attachment; filename="following-${site.slug}.csv"`);
@@ -1302 +1302 @@
 });
 
-router.post('/news/following/import', requireSiteManager, express.urlencoded({ extended: false, limit: '256kb' }), async (req, res) => {
-  const site = res.locals.site;
-  const csv = (req.body && req.body.csv) || '';
-  if (!site || !String(csv).trim()) return res.redirect('/following?error=' + encodeURIComponent('Geen lijst ontvangen'));
+// Een bestand OF geplakte tekst. Multer leest een multipart-formulier, en dat
+// bevat allebei: het bestandsveld en het tekstveld. In het geheugen, niet op
+// schijf: dit is een lijstje adressen van een paar kilobyte dat na het lezen
+// niets meer te zoeken heeft op de server.
+const followingCsvUpload = multer({
+  storage: multer.memoryStorage(),
+  limits: { fileSize: 512 * 1024, files: 1 },
+}).single('csvfile');
+
+router.post('/news/following/import', requireSiteManager, followingCsvUpload, async (req, res) => {
+  const site = res.locals.site;
+  // Een geupload bestand wint van het plakveld: wie een bestand kiest bedoelt dat.
+  const csv = (req.file && req.file.buffer)
+    ? req.file.buffer.toString('utf8').replace(/^/, '')   // BOM eraf; Excel zet die erin
+    : ((req.body && req.body.csv) || '');
+  if (!site || !String(csv).trim()) return res.redirect('/connect?error=' + encodeURIComponent('Geen lijst ontvangen'));
 
   const { importFollowing } = await import('../services/ArchiveImportService.js');
@@ -1326 +1338 @@
     delen.push(`${r.mislukt.length} mislukt (${namen}${r.mislukt.length > 3 ? '…' : ''})`);
   }
-  res.redirect('/following?' + (r.mislukt.length ? 'error=' : 'success=') + encodeURIComponent(delen.join(', ')));
+  // Terug naar /connect: daar staat het blok, /following is de oude pagina.
+  res.redirect('/connect?' + (r.mislukt.length ? 'error=' : 'success=') + encodeURIComponent(delen.join(', ')));
 });