Changeset 931b4cb in Klonkt

Timestamp:

06/28/2026 12:22:06 PM (2 months ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

cca2e89

Parents:

0a58300

Message:

fix(proxy): tolerate leading double-slash paths (Apache [P] proxy)

A reverse proxy using 'RewriteRule ^{(.*)$ ​http://localhost:3000/$1' sends for the root and
path for sub-paths (the captured $1 keeps its leading slash), which matched no Express route
→ the whole site 404'd behind such a proxy. Collapse leading duplicate slashes early so Klonkt
is resilient to that common Apache .htaccess setup.}

File:

• src/server.js (modified) (1 diff)

src/server.js

@@ -149 +149 @@
 if (!isDev) app.set('trust proxy', 1);
 
+// Collapse leading duplicate slashes in the path. A reverse proxy that proxies with
+// `RewriteRule ^(.*)$ http://localhost:3000/$1` (Apache [P]) sends "//" for the root and
+// "//path" for sub-paths (the captured $1 keeps its leading slash) → Express matches no
+// route → the whole site 404'd behind such a proxy. Normalising here makes Klonkt resilient
+// to that common reverse-proxy setup. (Only the leading slashes; the query string is intact.)
+app.use((req, res, next) => {
+  if (req.url.startsWith('//')) req.url = req.url.replace(/^\/+/, '/');
+  next();
+});
+
 // Create/migrate the schema BEFORE anything touches the DB: the session store
 // queries the `sessions` table on construction, so on a fresh install the tables