Changeset 603d246 in Klonkt for src/routes/admin-paid.js

Timestamp:

07/21/2026 01:54:46 AM (7 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

c3d12a6

Parents:

e685f55

git-author:

Robin <roboburr@…> (07/21/2026 01:54:34 AM)

git-committer:

Robin <roboburr@…> (07/21/2026 01:54:46 AM)

Message:

Feature: show the exact Patreon redirect URI in the paid admin

Bart hit Patreon's own error page ("Redirect URI .../paid/callback is not
supported by client") because the redirect URI our OAuth flow sends was
never whitelisted in his Patreon client, and the admin page told him
nowhere what that URI is. Once we redirect to patreon.com with an
unregistered redirect_uri, Patreon refuses to send the visitor back (open
redirect protection) and shows its own JSON error, which we cannot skin.

The only real defence is correct setup, so the admin now shows the exact
redirect URI to paste into the Patreon client, with a copy button. The URI
is built the same way paid.js builds it (PUBLIC_BASE_URL or the request
host + /paid/callback), so they always match.

Changed files:
src/routes/admin-paid.js

• compute redirectUri (matches paid.js) and pass it to the view

src/views/pages/admin-paid.ejs

• "Zet deze redirect-URI in je Patreon-client" block + copy button

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

File:

• src/routes/admin-paid.js (modified) (2 diffs)

src/routes/admin-paid.js

@@ -18 +18 @@
 const router = express.Router();
 
+// The redirect URI the owner MUST whitelist in their Patreon client. Must match
+// exactly what paid.js sends, or Patreon shows its own error page (which we
+// cannot skin) instead of returning the visitor to us.
+const redirectUri = (req) =>
+  (process.env.PUBLIC_BASE_URL || `${req.protocol}://${req.get('host')}`).replace(/\/+$/, '') + '/paid/callback';
+
 function gate(req, res) {
   if (!premiumUnlocked()) {
@@ -34 +40 @@
     status: PaidPatreon.ownerStatus(res.locals.site.id),
     secretReady: cryptoBoxReady(),
+    redirectUri: redirectUri(req),
     saved: req.query.saved === '1',
     error: req.query.error || null,