Changeset 5f43245 in Klonkt

Timestamp:

06/16/2026 04:05:27 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

a742e9f

Parents:

00d54bc

Message:

Fix: fresh install crashes on first boot (sessions table doesn't exist yet)

initializeDatabase() was called AFTER the session-store constructor, but
SqliteSessionStore queries the sessions table during construction. On a
fresh DB (new self-host/instance) that table didn't exist yet ->
"no such table: sessions" -> crash-loop on the very first boot. Existing
instances already had their schema, so this went unnoticed.

initializeDatabase() is now moved to before the session middleware.

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

File:

• src/server.js (modified) (2 diffs)

src/server.js

@@ -125 +125 @@
 if (!isDev) app.set('trust proxy', 1);
 
+// Schema aanmaken/bijwerken VÓÓR iets de DB aanraakt: de session-store doet
+// bij constructie al een query op de `sessions`-tabel, dus bij een verse
+// install moeten de tabellen eerst bestaan (anders: "no such table: sessions"
+// → crash-loop op de allereerste boot).
+initializeDatabase();
+
 // Session middleware extracted into a variable so the WebSocket upgrade
 // handler can reuse it (it needs req.session to authenticate sockets).
@@ -147 +153 @@
 // (Verwijderd) TWA / digital-asset-links — alleen nodig voor de APK/TWA-variant.
 // Klonkt is PWA-only; geen assetlinks.json meer.
-
-initializeDatabase();
 
 // Cirkels: periodieke achtergrond-sync van remote instances (no-op tenzij tenancy='circle').