Changeset 94a876f in Klonkt

Timestamp:

06/18/2026 05:54:36 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

86793f9

Parents:

f735304

Message:

Hub: removed dead host-based site resolving (slug == hostname could never match)

Slugs may not contain dots (validation [a-z0-9_-]), so the "WHERE slug = host"
lookup never matched — dead code + an unnecessary DB query per hub request. Removed.
Real subdomain routing would match the subdomain label against the slug (separate
feature).

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

File:

• src/middleware/site.js (modified) (3 diffs)

src/middleware/site.js

@@ -1 +1 @@
 /**
  * Site middleware — resolve which site this request is for.
- * 
- * Resolution order:
- *   1. Path /sites/:slug → that site
- *   2. (Future) Subdomain bedrijf1.example.com → matching site
- *   3. Default site (first one in DB)
- * 
+ *
+ * Resolution order (hub-modus):
+ *   1. Pad /user/:slug → die site  (legacy /sites/:slug → 301 naar /user/)
+ *   2. Anders (solo, of hub-landing): de primaire/hoofd-site
+ *
  * Sets res.locals.site for all downstream handlers.
  */
@@ -29 +28 @@
   res.locals.tenancy = tenancy; // ook beschikbaar voor views
 
-  // In HUB-mode mapt /sites/:slug en (later) een subdomein naar een specifieke
-  // site. In SOLO-mode bestaat er maar één site: we slaan die routing over en
-  // pinnen altijd op de primaire site.
+  // In HUB-mode mapt /user/:slug naar een specifieke site. In SOLO-mode bestaat
+  // er maar één site: we slaan die routing over en pinnen op de primaire site.
   if (tenancy === 'hub') {
     // Een Klonkt-site is canoniek bereikbaar via /user/:slug. /sites/:slug is een
@@ -49 +47 @@
       }
     }
-    const host = req.get('host')?.toLowerCase().replace(/:\d+$/, '');
-    if (host) {
-      const site = db.prepare('SELECT * FROM sites WHERE slug = ?').get(host);
-      if (site) {
-        res.locals.site = site;
-        res.locals.siteUrlBase = '';
-        return next();
-      }
-    }
+    // (Verwijderd: een dode "slug == hostname"-subdomein-hack. Slugs mogen geen
+    // punten bevatten, dus die kon nooit matchen. Echte subdomein-routing zou de
+    // subdomein-LABEL tegen de slug matchen — een aparte feature, niet dit.)
   }