Changeset 2fc9b7f in Klonkt

Timestamp:

06/20/2026 03:14:25 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

389aa99

Parents:

7990e00

Message:

fix(circle): do not add hubs/non-circle sites as partner

A hub does not federate (its /.klonkt/actor.json returns 404), but it was
still added to the circle with a dead "error / HTTP 404" row. The add now
rolls back the insert on a 404 and shows a clear message ("a hub cannot be
a circle partner"). Temporary (non-404) errors still leave the link so it
can be refreshed later.

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

File:

• src/routes/admin-circle.js (modified) (1 diff)

src/routes/admin-circle.js

@@ -69 +69 @@
     return res.redirect('/admin/circle?success=' + encodeURIComponent('Toegevoegd en gesynchroniseerd ✓'));
   } catch (e) {
+    const msg = String((e && e.message) || e);
+    // 404 = geen cirkel-endpoint. Hubs federeren bewust NIET (hun /.klonkt/actor.json
+    // geeft 404), net als losse niet-Klonkt-sites. Niet toevoegen: rol de insert terug
+    // zodat er geen dode "fout"-rij in de cirkel blijft staan.
+    if (/\b404\b/.test(msg)) {
+      db.prepare('DELETE FROM circle_links WHERE id = ?').run(id);
+      return res.redirect('/admin/circle?error=' + encodeURIComponent('Niet toegevoegd: deze site doet niet mee aan cirkels. Een hub kan geen cirkel-partner zijn (en losse/niet-Klonkt-sites ook niet).'));
+    }
+    // Andere (mogelijk tijdelijke) fout → link blijft staan; later "Verversen".
     return res.redirect('/admin/circle?success=' + encodeURIComponent('Toegevoegd — synchroniseren mislukte (klik "Verversen" om opnieuw te proberen)'));
   }