Changeset 11ba289 in Klonkt

Timestamp:

06/20/2026 01:20:57 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

6d882b1

Parents:

36afccd

Message:

fix(auth): Google login matches on linked google_sub first (not email)

An admin who linked their Google account to an account with a different
email address (e.g. demo-admin demo@… + roboburr@…) was
not recognized on Google login (lookup by email) → a new member account
was created instead. Now looks up google_sub first → logs in with the
proper (admin) role, regardless of email address. Falls back to email for
unlinked listeners.

Co-Authored-By: Claude <noreply@…>

File:

• src/routes/auth.js (modified) (1 diff)

src/routes/auth.js

@@ -335 +335 @@
     if (!email || info.email_verified === false) return failLogin('email');
 
-    let user = db.prepare('SELECT * FROM users WHERE LOWER(email) = ?').get(email);
-
-    if (user && (user.role === 'god' || user.role === 'admin')) {
-      // Beheerder mag ALLEEN met Google in als 'ie z'n Google expliciet gekoppeld
-      // heeft (matchende google_sub). Anders blijft gelden: Google = nooit beheer.
-      if (!(user.google_sub && info.sub && user.google_sub === info.sub)) {
+    // Zoek EERST op de gekoppelde Google-account (google_sub). Een sub-match is het
+    // expliciete koppel-bewijs → log in met de eigen rol, OOK als het Google-
+    // mailadres afwijkt van het account-mailadres (bv. een beheerder die een ander
+    // Gmail koppelt). Daarna pas op e-mail.
+    let user = info.sub ? db.prepare('SELECT * FROM users WHERE google_sub = ?').get(info.sub) : null;
+
+    if (user) {
+      // Gekoppeld account gevonden → eigen rol behouden. Avatar bijwerken indien leeg.
+      db.prepare(`
+        UPDATE users SET avatar_url = COALESCE(avatar_url, ?), updated_at = CURRENT_TIMESTAMP WHERE id = ?
+      `).run(info.picture || null, user.id);
+    } else {
+      user = db.prepare('SELECT * FROM users WHERE LOWER(email) = ?').get(email);
+      if (user && (user.role === 'god' || user.role === 'admin')) {
+        // Beheerder gevonden op e-mail maar ZONDER gekoppelde sub → Google geeft
+        // nooit beheer. Eerst koppelen via Account → Inloggen met Google.
         return failLogin('admin');
+      } else if (user) {
+        // Bestaande luisteraar: koppel google_sub/avatar als die ontbreken.
+        db.prepare(`
+          UPDATE users SET google_sub = COALESCE(google_sub, ?), avatar_url = COALESCE(avatar_url, ?),
+            updated_at = CURRENT_TIMESTAMP WHERE id = ?
+        `).run(info.sub || null, info.picture || null, user.id);
+      } else {
+        // Nieuwe luisteraar — altijd member.
+        const userId = uuid();
+        const username = uniqueUsername(info.name || email.split('@')[0]);
+        db.prepare(`
+          INSERT INTO users (id, username, email, password_hash, role, avatar_url, theme, palette, google_sub)
+          VALUES (?, ?, ?, '!google-oauth', 'member', ?, 'dark', 'sage', ?)
+        `).run(userId, username, info.email || email, info.picture || null, info.sub || null);
+        user = db.prepare('SELECT * FROM users WHERE id = ?').get(userId);
       }
-      // gekoppeld + match → doorgaan met de eigen (beheer)rol.
-    } else if (user) {
-      // Bestaande luisteraar: koppel google_sub/avatar als die ontbreken; weiger
-      // als al aan een ander Google-account gekoppeld.
-      if (user.google_sub && info.sub && user.google_sub !== info.sub) return failLogin('linked');
-      db.prepare(`
-        UPDATE users SET google_sub = COALESCE(google_sub, ?), avatar_url = COALESCE(avatar_url, ?),
-          updated_at = CURRENT_TIMESTAMP WHERE id = ?
-      `).run(info.sub || null, info.picture || null, user.id);
-    } else {
-      // Nieuwe luisteraar — altijd member.
-      const userId = uuid();
-      const username = uniqueUsername(info.name || email.split('@')[0]);
-      db.prepare(`
-        INSERT INTO users (id, username, email, password_hash, role, avatar_url, theme, palette, google_sub)
-        VALUES (?, ?, ?, '!google-oauth', 'member', ?, 'dark', 'sage', ?)
-      `).run(userId, username, info.email || email, info.picture || null, info.sub || null);
-      user = db.prepare('SELECT * FROM users WHERE id = ?').get(userId);
     }