Changeset 00d54bc in Klonkt

Timestamp:

06/16/2026 03:38:33 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

5f43245

Parents:

baa2e59

Message:

Fix: htmx-partial 500 on non-ASCII characters in post title

The HX-Trigger-After-Settle header received the post title unfiltered via
JSON.stringify. HTTP header values are Latin-1; an em-dash, smart quote,
or emoji in the title (e.g. "Welkom — gebouwd met Klonkt") caused setHeader
to throw ERR_INVALID_CHAR -> partial 500 -> the card appeared "not clickable"
(full page worked fine, as it doesn't set the header).

Escape non-ASCII to \uXXXX: the header stays ASCII-safe and valid JSON that
htmx reads back unchanged.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@…>

File:

• src/middleware/render.js (modified) (2 diffs)

src/middleware/render.js

@@ -95 +95 @@
 
     if (isPartial) {
-      // HTMX: just send the content. Set HX-Trigger for body class swap
-      res.setHeader('HX-Trigger-After-Settle', JSON.stringify({
+      // HTMX: just send the content. Set HX-Trigger for body class swap.
+      // HTTP-header values are Latin-1 only — a title with an em-dash, smart
+      // quote or emoji (e.g. "Welkom — gebouwd met Klonkt") would make
+      // setHeader throw ERR_INVALID_CHAR and 500 the partial, so the card
+      // looks "unclickable". Escape any non-ASCII to \uXXXX: the header stays
+      // ASCII-safe and remains valid JSON that htmx parses back unchanged.
+      const triggerJson = JSON.stringify({
         pcmsNav: { bodyClass: locals.bodyClass },
         pcmsPostSwap: data.post ? {
@@ -103 +108 @@
           pageTitle: locals.pageTitle,
         } : null,
-      }));
+      }).replace(/[€-￿]/g, (ch) => '\\u' + ch.charCodeAt(0).toString(16).padStart(4, '0'));
+      res.setHeader('HX-Trigger-After-Settle', triggerJson);
       return res.send(pageContent);
     }