Changeset 3ceaef6 in Klonkt for src/routes/search.js

Timestamp:

09/30/2026 06:16:32 AM (3 hours ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

9eb6e1b

Parents:

407353e

Message:

Zoeken op een adres: het object erachter, met de knoppen erbij

Plak je een link naar een fediverse-post of -profiel in de zoekbalk, dan is dat
geen zoekterm maar een aanwijzing. Klonkt haalt hem op en zet hem bovenaan de
resultaten, met dezelfde knoppen als /authorize_interaction: antwoorden,
waarderen, boosten, stemmen, volgen en melden.

De weg erheen was vooral opruimen. De render en de formulieren stonden als 200
regels in authorize-interaction.ejs, en er een tweede kopie van maken voor de
zoekpagina zou een tweede plek zijn waar een poort, een sanitizer of een knop
vergeten kan worden. Ze staan nu in partials/remote-object.ejs en
partials/remote-actor.ejs, met de opmaak in een derde. De pagina die ze had
gebruikt ze nu ook, dus er is één render en de formulieren posten in beide
gevallen naar /authorize_interaction/..., waar het werk en de rechtencontrole
al zaten.

DRIE DINGEN DIE VASTLIGGEN, en elk om een eigen reden:

Het schema is verplicht. "soundfabrics.nl" is een geldige zoekterm -- iemand
zoekt naar de naam -- en een kale domeinnaam als adres opvatten maakt van elke
zoekopdracht met een punt erin een netwerkverzoek. Met ​https:// ervoor is er
geen twijfel over de bedoeling.

Ophalen mag alleen een beheerder van de site laten doen. Dat is dezelfde grens
als /authorize_interaction, en met opzet: ophalen is een uitgaand verzoek dat
deze server namens de bezoeker doet. Zonder die grens is de zoekbalk een
haalservice waarmee een vreemde deze server laat aankloppen op adressen naar
keuze. Een bezoeker die toch een link plakt krijgt de gewone zoekresultaten en
een zin die zegt waarom er verder niets staat.

De ophaling gaat ONDERTEKEND als de site. Een post die alleen voor volgers
zichtbaar is weigert een anonieme GET, en dan lijkt een bestaande post te
ontbreken -- dezelfde val als bij het antwoorden op een post van een vriend.

Bijgekomen zonder dat erom gevraagd is, en het is het vermelden waard: het
profiel toont nu zijn avatar. Die hadden we al in handen (actor_icon uit
resolveRemoteActor) en hij stond op geen van beide schermen.

Getoetst: de herkenning zelf, dat een zoekterm GEEN verzoek wordt, dat de post
met zijn vier formulieren binnenkomt, dat een profiel de volgknop geeft, dat
een bezoeker niets laat ophalen, en dat /authorize_interaction na de verhuizing
nog steeds hetzelfde toont -- die pagina had zelf geen enkele toets, dus dat
was de bodem die er onder moest. Tegenproef per zaak.

Nog open: een @naam@server in de zoekbalk (webfinger kan het al, dus klein
werk) en een hint in de live-suggesties terwijl je plakt.

shaer-utpi

File:

• src/routes/search.js (modified) (6 diffs)

src/routes/search.js

@@ -9 +9 @@
  *   4. Pages (Agenda / Downloads / Links / Press kit / Archive) by name — only
  *      the available ones.
+ *   5. Een ADRES van een andere server (shaer-utpi): plak je een link naar een
+ *      fediverse-post of -profiel, dan is dat geen zoekterm maar een aanwijzing.
+ *      Die halen we op en tonen we, met dezelfde knoppen als
+ *      /authorize_interaction: antwoorden, waarderen, boosten, stemmen,
+ *      volgen, melden.
  *
  * FTS5: user input is tokenised on non-letter/digit chars and each token is wrapped
@@ -21 +26 @@
 import { premiumUnlocked } from '../services/PatreonService.js';
 import { t as i18nT, resolveLang } from '../services/i18n.js';
+import ActivityPubService from '../services/ActivityPubService.js';
+import PermissionsService from '../services/PermissionsService.js';
 
 const router = express.Router();
@@ -45 +52 @@
   if (!s || /^[…\s]*$/.test(s)) return esc((excerpt || '').slice(0, 160));
   return s;
+}
+
+/**
+ * Is dit een ADRES en geen zoekterm? Dan het genormaliseerde adres, anders null.
+ *
+ * HET SCHEMA IS VERPLICHT, en dat is een keuze. "soundfabrics.nl" is een
+ * geldige zoekterm -- iemand zoekt naar de naam -- en een bare domeinnaam als
+ * adres opvatten maakt van elke zoekopdracht met een punt erin een
+ * netwerkverzoek. Met https:// ervoor is er geen twijfel over de bedoeling.
+ *
+ * De fragmentverwijzing gaat eraf: die hoort bij de browser, niet bij het
+ * object, en met #comment eraan is het een andere sleutel voor hetzelfde ding.
+ */
+export function lookupUri(q) {
+  const s = String(q || '').trim();
+  if (!/^https?:\/\//i.test(s) || /\s/.test(s)) return null;
+  try {
+    const u = new URL(s);
+    if (u.protocol !== 'http:' && u.protocol !== 'https:') return null;
+    // Een naam zonder punt is geen adres op het open net (localhost, een
+    // intern hostname). De SSRF-bewaking in de transportlaag houdt de rest
+    // tegen; dit scheelt het verzoek.
+    if (!u.hostname.includes('.')) return null;
+    u.hash = '';
+    return u.toString();
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * Mag deze bezoeker een adres laten OPHALEN?
+ *
+ * Dezelfde grens als /authorize_interaction (requireSiteManager), en met opzet:
+ * ophalen is een uitgaand verzoek dat deze server namens iemand anders doet.
+ * Voor een willekeurige bezoeker zou de zoekbalk daarmee een haalservice zijn
+ * waarmee je andermans server kunt laten aankloppen op adressen naar keuze.
+ *
+ * Dit spiegelt requireSiteManager in plaats van hem aan te roepen: die stuurt
+ * je naar het inlogscherm, en een zoekopdracht hoort niemand weg te sturen.
+ */
+function mayLookUp(req, res) {
+  const u = req.session && req.session.user;
+  if (!u) return false;
+  if (u.role === 'god' || u.role === 'kijker') return true;
+  const site = res.locals.site;
+  return !!(site && PermissionsService.canAdminSite(u, site));
 }
 
@@ -143 +197 @@
 
 // ── Full results page ────────────────────────────────────────────────────────
-router.get('/', (req, res) => {
+router.get('/', async (req, res) => {
   const site = res.locals.site;
   if (!site) return res.status(404).send('No site');
@@ -153 +207 @@
       results: [], tracks: [], events: [], pages: [], total: 0,
     });
+  }
+
+  // Een adres wordt OPGEHAALD, niet doorzocht (shaer-utpi). Eerst als post,
+  // dan als profiel: een URL die geen van beide oplevert is gewoon een
+  // zoekterm die toevallig op een link lijkt, en dan blijft de rest staan.
+  const remoteUri = lookupUri(rawQ);
+  const mayLookup = remoteUri ? mayLookUp(req, res) : false;
+  let remote = null;
+  let remoteKind = null;
+  if (remoteUri && mayLookup) {
+    // Ondertekend als deze site: een post die alleen voor volgers zichtbaar is
+    // weigert een anonieme GET, en dan lijkt een bestaande post te ontbreken.
+    try { remote = await ActivityPubService.resolveRemoteNote(remoteUri, { asSlug: site.slug }); } catch { /* onbereikbaar */ }
+    if (remote) remoteKind = 'note';
+    if (!remote) {
+      try { remote = await ActivityPubService.resolveRemoteActor(remoteUri); } catch { /* onbereikbaar */ }
+      if (remote) remoteKind = 'actor';
+    }
   }
 
@@ -161 +233 @@
     results: r.results, tracks: r.tracks, events: r.events, pages: r.pages,
     total, queryError: r.queryError,
+    remoteUri, remote, remoteKind, mayLookup,
+    remoteReacted: (remoteKind === 'note') ? ActivityPubService.getReaction(site.slug, remote.object_uri || remoteUri) : { liked: false, boosted: false },
+    siteTitle: site.title || '',
   });
 });