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


Ignore:
Timestamp:
08/18/2026 06:50:08 PM (3 weeks ago)
Author:
Bart <bart@…>
Branches:
main
Children:
d85b66f
Parents:
9da8027
git-author:
Bart <bart@…> (08/18/2026 06:05:22 PM)
git-committer:
Bart <bart@…> (08/18/2026 06:50:08 PM)
Message:

Gastlogin via OpenWebAuth: een fan is een volger, geen accounthouder

fan_only betekende altijd al "mijn volgers op de fediverse", maar de poort vroeg
om een KLONKT-ACCOUNT. Dat is de verkeerde vraag, en hij sloot precies de mensen
buiten voor wie de poort openstond. Nu kan een bezoeker bij zijn EIGEN server
bewijzen dat hij @iemand@ergens is (FEP-61cf), en volgt hij deze site, dan is
hij binnen. Geen account hier, geen wachtwoord hier, geen cookie van een derde.

Wij zijn alleen de TARGET instance. Dat is de prettige helft: de home instance
heeft prive-sleutels nodig, wij alleen publieke. Er staat hier dus geen geheim
van iemand anders. De /magic-kant (Klonkt-gebruikers laten inloggen OP andere
sites) is bewust niet gebouwd -- andere functie.

De handtekening-verificatie is NIET opnieuw geschreven: AP.verifyRequest() doet
dit al voor de inbox, inclusief het vastpinnen van de sleutel op de herkomst van
de actor, een replay-venster en een verplichte digest. Een tweede implementatie
van "is deze aanvraag echt van wie hij zegt" is precies wat je niet wilt.

De drie aanvallen die de FEP noemt, hebben elk een toets:

  • IMPERSONATIE: ?zid= bepaalt niets, alleen het ingewisselde ?owt= telt. Mallory kan een link maken met zid=bob, maar komt terug met een token dat Mallory zegt.
  • OPEN REDIRECT: het ontdekte endpoint moet dezelfde host hebben als het adres dat de bezoeker intypte.
  • DoS: tokens vervallen in minuten, gaan na een keer gebruiken weg, en elke uitgifte veegt de oude op.

Onderweg gemeten en vastgelegd: PKCS#1 v1.5 GOOIT GEEN FOUT bij een verkeerde
sleutel. OpenSSL 3 doet aan implicit rejection en geeft afgeleide onzin terug,
juist zodat niemand aan het foutgedrag kan aflezen of zijn gok klopte. 200
vreemde sleutels: 0 fouten, 0 keer het token. De toets test dus "er komt iets
anders uit", niet "het knalt" -- anders schrijft de volgende lezer weer een
assert.throws die per ongeluk slaagt.

Webfinger op de eigen wortel wijst een home instance naar /owa/token. Alleen
origin + '/'; een ACTOR-uri met een pad blijft een 400, want dat legt
webfinger-bare-host.test.js vast en die keuze draai ik niet om als bijvangst.

De fanpoort toont nu het adresveld als hoofdweg en de lokale inlog als tweede,
en fgate.sub zegt niet langer "ingelogde vrienden" maar wat de poort werkelijk
vraagt.

1135 toetsen groen (was 1106). End-to-end nagelopen op een KOPIE van de
database: token inwisselen zet de sessie en haalt het token uit de URL, een
bewezen volger krijgt de tekst, een bewezen niet-volger krijgt de poort en geen
byte van de inhoud, en anoniem idem.

Co-Authored-By: Claude Opus 5 <claude@…>

File:
1 edited

Legend:

Unmodified
Added
Removed
  • src/routes/posts.js

    r9da8027 raf2cc73  
    2525import { verifyBlob } from '../services/CryptoBox.js';
    2626import { postEntry } from '../services/PostAccessService.js';
     27import * as OWA from '../services/OpenWebAuthService.js';
    2728import MusicMeta from '../services/MusicMeta.js';
    2829import { mediaDir } from '../config/paths.js';
     
    702703// tekst op, want wat niet gerenderd wordt kan ook niet lekken.
    703704function readerItems(site, rows, req) {
    704   const viewer = { user: req.session?.user || null, site, unlockedSlug: null };
     705  const viewer = OWA.viewerFor(req, site, { unlockedSlug: null });
    705706  return rows.map((post) => ({
    706707    post,
     
    15891590  // Anonymous visitors get a clean login gate instead of the content (the title/
    15901591  // teaser may still appear elsewhere as a teaser).
    1591   if (post.fan_only && !(req.session && req.session.user)) {
     1592  // Een bezoeker die via OpenWebAuth bewees @iemand@ergens te zijn EN deze site
     1593  // volgt, is precies wie fan_only bedoelde. Die hoeft geen poort te zien.
     1594  const _fediVolger = OWA.isFollowerOf(site.slug, OWA.guestActor(req));
     1595  if (post.fan_only && !(req.session && req.session.user) && !_fediVolger) {
    15921596    // Same Newer/Older navigation as on a normal post, so the visitor doesn't get
    15931597    // stuck on the fan gate but can keep browsing.
     
    15981602      fgTitle: post.title || '',
    15991603      fgNext: (res.locals.siteUrlBase || '') + '/' + post.slug,
     1604      owaError: !!(req.query && req.query.owa_error),
    16001605      newerPost,
    16011606      olderPost,
Note: See TracChangeset for help on using the changeset viewer.