Changeset ca507ef in Klonkt for src/routes/admin-settings.js

Timestamp:

06/19/2026 08:29:28 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

7eb35ae

Parents:

977f65d

Message:

feat: SMTP configurable via Admin → Settings (app_settings, env fallback) + test email

• config/mailer.js: reads SMTP from app_settings (smtp_host/port/user/pass/from), with env as fallback; transport rebuilds automatically on change (no restart needed); mailerStatus() for the UI (without leaking the password).
• admin-settings: POST /smtp (save; password only updated when a new value is given; clear) + POST /smtp/test (test email). GET includes smtp status.
• admin-settings.ejs: Email (SMTP) card with host/port/user/pass/from + save/ clear + test-email button (only shown when configured).

Once SMTP is filled in, newsletter sending, show-notify and double opt-in work.

Co-Authored-By: Claude <noreply@…>

File:

• src/routes/admin-settings.js (modified) (3 diffs)

src/routes/admin-settings.js

@@ -22 +22 @@
 import { requireGod } from '../middleware/auth.js';
 import { getTenancy, setTenancy, getSetting, setSetting } from '../services/SettingsService.js';
+import { mailerStatus, sendMail } from '../config/mailer.js';
 import { entitlementStatus, premiumUnlocked } from '../services/PatreonService.js';
 import { googleConfigured, redirectUri, currentClientId, clientSecretSet } from '../config/google.js';
@@ -83 +84 @@
       secretSet: clientSecretSet(),
     },
+    smtp: mailerStatus(),
     success: req.query.success || null,
     error: req.query.error || null,
@@ -151 +153 @@
 });
 
+// ── SMTP / e-mail-instellingen ────────────────────────────────────
+router.post('/smtp', requireGod, (req, res) => {
+  const b = req.body || {};
+  if (b.clear === '1') {
+    ['smtp_host', 'smtp_port', 'smtp_user', 'smtp_pass', 'smtp_from'].forEach((k) => setSetting(k, ''));
+    return res.redirect('/admin/settings?success=' + encodeURIComponent('SMTP-instellingen gewist'));
+  }
+  setSetting('smtp_host', (b.smtp_host || '').toString().trim());
+  setSetting('smtp_port', (b.smtp_port || '').toString().trim());
+  setSetting('smtp_user', (b.smtp_user || '').toString().trim());
+  setSetting('smtp_from', (b.smtp_from || '').toString().trim());
+  // Wachtwoord alleen overschrijven als er een nieuwe waarde is ingevoerd.
+  const pass = (b.smtp_pass || '').toString();
+  if (pass) setSetting('smtp_pass', pass);
+  res.redirect('/admin/settings?success=' + encodeURIComponent('SMTP-instellingen opgeslagen'));
+});
+
+// Testmail sturen naar een opgegeven adres (of de ingelogde gebruiker).
+router.post('/smtp/test', requireGod, async (req, res) => {
+  const to = ((req.body && req.body.to) || (req.session.user && req.session.user.email) || '').toString().trim();
+  if (!to || to.indexOf('@') === -1) {
+    return res.redirect('/admin/settings?error=' + encodeURIComponent('Geef een geldig test-e-mailadres op.'));
+  }
+  try {
+    await sendMail({
+      to,
+      subject: 'Klonkt — SMTP-test',
+      text: 'Gelukt! Je SMTP-instellingen werken. Dit is een testbericht van je Klonkt-site.',
+      html: '<p>Gelukt! Je <strong>SMTP-instellingen werken</strong>. Dit is een testbericht van je Klonkt-site.</p>',
+    });
+    res.redirect('/admin/settings?success=' + encodeURIComponent('Testmail verstuurd naar ' + to));
+  } catch (e) {
+    res.redirect('/admin/settings?error=' + encodeURIComponent('Testmail mislukt: ' + (e.message || e)));
+  }
+});
+
 export default router;