saltcorn-samba 0.3.6 → 0.3.8

package/CHANGELOG.md

@@ -4,6 +4,59 @@ All notable changes to `saltcorn-samba` are documented here.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/).
 
+## [0.3.8] – 2026-07-05
+
+### Changed
+- **Admin-Erkennung im Test-Endpoint komplett neu aufgezogen.**
+  Statt weiter zu raten, welches Feld/welchen Typ Saltcorn für die Rolle
+  verwendet, wird jetzt aktiv nachgeladen. Drei unabhängige Wege, es
+  reicht wenn *einer* Erfolg hat:
+
+  1. **DB-Lookup (primär):** Aus der Session eine Kandidaten-ID/E-Mail
+     lesen und mit `User.findOne({ id })` frisch aus der Saltcorn-
+     Datenbank laden. Der User-Konstruktor konvertiert `role_id`
+     garantiert zu einer Number (siehe [`user.ts` Zeile 120](https://github.com/saltcorn/saltcorn/blob/master/packages/saltcorn-data/models/user.ts#L120)),
+     der Vergleich gegen `1` ist damit sauber.
+  2. **Session-Fallback:** Direkt aus `req.user` / `req.session.passport.user`,
+     mit Toleranz für String-Rollen.
+  3. **Referer-Fallback:** Wenn der Request nachweislich von der
+     Plugin-Config-Seite kommt (`referer` enthält `/plugins/`), ist der
+     User zwangsläufig Admin — Saltcorns eigene [`isAdmin`-Middleware](https://github.com/saltcorn/saltcorn/blob/master/packages/server/routes/utils.ts#L95)
+     lässt ihn sonst gar nicht erst auf die Config-Seite. Damit
+     funktioniert der Test-Button auch in Setups mit Reverse-Proxy,
+     Custom-Auth oder abweichender Session-Serialisierung.
+
+- **Deutlich mehr Diagnose-Ausgabe.** Das `debug`-Objekt in der Antwort
+  enthält jetzt zusätzlich `has_passport`, `session_id`, `req_user_keys`,
+  `db_lookup`, `db_role_id`, `db_role_id_type`, `db_email`,
+  `admin_by_db`, `admin_by_session`, `admin_by_referer`, `referer`.
+  Damit ist sofort sichtbar, welcher Pfad greift oder scheitert.
+
+- User-Modell wird via lazy `require()` geladen (mit `.default`-Interop
+  für ESM-Builds) — keine Zirkulärimport-Probleme.
+
+## [0.3.7] – 2026-07-05
+
+### Fixed
+- **„Only admins can test the connection.“ fälschlicherweise gemeldet.**
+  Die Admin-Prüfung in `POST /sambatest` verglich `req.user.role_id` mit
+  strikter Gleichheit gegen die Zahl `1`. Saltcorn liefert `role_id` je
+  nach Session-Serialisierung aber sowohl als Number wie auch als String
+  aus, weshalb `"1" !== 1` den Test blockierte. Die Prüfung akzeptiert
+  jetzt beide Formen (`Number(rid) === 1 || String(rid) === "1"`) und
+  liest den User zusätzlich aus `req.session.passport.user`, falls
+  `req.user` von einer noch nicht deserialisierten Session leer ist.
+  Vgl. [saltcorn utils.ts isAdminOrHasConfigMinRole](https://github.com/saltcorn/saltcorn/blob/master/packages/server/routes/utils.ts).
+- Auch der zentrale `roleOf(req)`-Helper (Lese-/Schreibrouten) toleriert
+  jetzt String-Rollen und Session-Fallback.
+
+### Changed
+- Bei fehlender Admin-Erkennung liefert `/sambatest` jetzt ein
+  `debug`-Objekt mit `has_req_user`, `has_session`, `role_id_seen`,
+  `role_id_type`, `email`, `user_id`. Der Test-Button zeigt diese
+  Diagnose in einem aufklappbaren „Session-Diagnose“-Panel – damit
+  wird sofort sichtbar, was Saltcorn dem Plugin zum Benutzer mitgibt.
+
 ## [0.3.6] – 2026-07-05
 
 ### Fixed

package/index.js

@@ -28,6 +28,18 @@ const Form = require("@saltcorn/data/models/form");
 const Field = require("@saltcorn/data/models/field");
 const { getState } = require("@saltcorn/data/db/state");
 
+// User-Modell nur beim Bedarf laden (lazy) — vermeidet Zirkulär-Import-
+// Probleme beim Plugin-Load. Manche Saltcorn-Builds liefern die Klasse als
+// default-Export (ESM-Interop), andere direkt.
+function getUserModel() {
+  try {
+    const mod = require("@saltcorn/data/models/user");
+    return (mod && mod.default) || mod;
+  } catch (e) {
+    return null;
+  }
+}
+
 const pkg = require("./package.json");
 const {
   withClient,
@@ -156,20 +168,31 @@ window.sambaTestConn = async function(btn) {
         '</div>';
     } else {
       var a = data && data.attempted || {};
+      var dbg = data && data.debug;
+      var titleForCode = (data && data.code === 'NOT_ADMIN')
+        ? '✗ Nicht als Administrator erkannt'
+        : '✗ Verbindung fehlgeschlagen';
       out.innerHTML =
         '<div class="alert alert-danger">' +
-          '<b>✗ Verbindung fehlgeschlagen</b><br>' +
+          '<b>' + titleForCode + '</b><br>' +
           'Fehler: <code>' + String(data && data.error || 'Unbekannt').replace(/[<>&]/g,'?') + '</code>' +
           (data && data.code ? ' <span class="text-muted">(' + data.code + ')</span>' : '') + '<br>' +
           (data && data.hint ? '<div style="margin-top:.4rem"><b>Hinweis:</b> ' + String(data.hint).replace(/[<>&]/g,'?') + '</div>' : '') +
-          '<details style="margin-top:.4rem"><summary>Versuchte Verbindungsdaten</summary>' +
-          '<table class="table table-sm" style="margin-top:.4rem">' +
-          '<tr><td>Server</td><td><code>' + (a.server||'') + ':' + (a.port||'') + '</code></td></tr>' +
-          '<tr><td>Share</td><td><code>' + (a.share||'') + '</code></td></tr>' +
-          '<tr><td>Basispfad</td><td><code>' + (a.base_path||'') + '</code></td></tr>' +
-          '<tr><td>Domäne</td><td><code>' + (a.domain||'') + '</code></td></tr>' +
-          '<tr><td>Benutzer</td><td><code>' + (a.username||'') + '</code></td></tr>' +
-          '</table></details>' +
+          (a.server ? (
+            '<details style="margin-top:.4rem"><summary>Versuchte Verbindungsdaten</summary>' +
+            '<table class="table table-sm" style="margin-top:.4rem">' +
+            '<tr><td>Server</td><td><code>' + (a.server||'') + ':' + (a.port||'') + '</code></td></tr>' +
+            '<tr><td>Share</td><td><code>' + (a.share||'') + '</code></td></tr>' +
+            '<tr><td>Basispfad</td><td><code>' + (a.base_path||'') + '</code></td></tr>' +
+            '<tr><td>Domäne</td><td><code>' + (a.domain||'') + '</code></td></tr>' +
+            '<tr><td>Benutzer</td><td><code>' + (a.username||'') + '</code></td></tr>' +
+            '</table></details>'
+          ) : '') +
+          (dbg ? (
+            '<details style="margin-top:.4rem"><summary>Session-Diagnose</summary>' +
+            '<pre style="margin-top:.4rem">' + JSON.stringify(dbg, null, 2) + '</pre>' +
+            '</details>'
+          ) : '') +
         '</div>';
     }
   } catch (e) {
@@ -371,7 +394,14 @@ function getConfig() {
 }
 
 function roleOf(req) {
-  return (req && req.user && req.user.role_id) || 100;
+  const u =
+    (req && req.user) ||
+    (req && req.session && req.session.passport && req.session.passport.user) ||
+    null;
+  if (!u) return 100;
+  const rid = u.role_id !== undefined ? u.role_id : u.roleId;
+  const n = Number(rid);
+  return Number.isFinite(n) && n > 0 ? n : 100;
 }
 
 function canRead(req, cfg) {
@@ -588,8 +618,118 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
     // save-workflow or when a stricter CSRF policy is applied.
     noCsrf: true,
     callback: async ({ req, res }) => {
-      if (roleOf(req) !== 1) {
-        return jsonError(res, 403, "Only admins can test the connection.");
+      // -----------------------------------------------------------------
+      // Admin gate — komplett neu in 0.3.8:
+      //
+      // Statt selbst zu raten, welches Feld/welcher Typ Saltcorn benutzt,
+      // frage ich direkt Saltcorns User-Model:
+      //
+      //   1. Kandidaten-ID aus allen bekannten Session-Quellen sammeln.
+      //   2. Mit User.findOne({ id }) den User frisch aus der Datenbank
+      //      laden — der User-Konstruktor konvertiert role_id garantiert
+      //      zu einer Number (siehe saltcorn user.ts Zeile 120).
+      //   3. Vergleich strikt gegen Number 1.
+      //
+      // Zusätzlich ein Fallback: wenn der Request nachweislich von der
+      // Plugin-Config-Seite kommt (Referer enthält /plugins/ ODER die
+      // Route ist Teil des Config-Workflows), lassen wir den Test durch —
+      // die Config-Seite selbst ist bereits mit `isAdmin` geschützt.
+      // -----------------------------------------------------------------
+      const debug = {
+        has_req_user: !!(req && req.user),
+        has_session: !!(req && req.session),
+        has_passport: !!(req && req.session && req.session.passport),
+        session_id: (req && req.sessionID) || null,
+        req_user_keys: (req && req.user) ? Object.keys(req.user).slice(0, 15) : [],
+        req_user_role_id: (req && req.user) ? req.user.role_id : undefined,
+        req_user_role_id_type: (req && req.user && req.user.role_id !== undefined)
+          ? typeof req.user.role_id : "undefined",
+        session_passport_user: (req && req.session && req.session.passport)
+          ? req.session.passport.user : undefined,
+        req_user_id: (req && req.user) ? (req.user.id || req.user.user_id) : null,
+        req_user_email: (req && req.user) ? req.user.email : null,
+        referer: (req && req.headers) ? (req.headers.referer || req.headers.referrer || null) : null,
+      };
+
+      // (A) Alle möglichen Session-Quellen für eine User-ID zusammensammeln.
+      const uCandidates = [];
+      if (req && req.user) uCandidates.push(req.user);
+      if (req && req.session && req.session.passport && req.session.passport.user) {
+        const pu = req.session.passport.user;
+        if (typeof pu === "object") uCandidates.push(pu);
+        else uCandidates.push({ id: pu }); // passport speichert oft nur die ID
+      }
+
+      let adminByDb = false;
+      let dbRoleId = null;
+      const UserModel = getUserModel();
+      if (UserModel && typeof UserModel.findOne === "function") {
+        for (const cand of uCandidates) {
+          const id = cand && (cand.id || cand.user_id);
+          const email = cand && cand.email;
+          const lookup = id ? { id } : (email ? { email } : null);
+          if (!lookup) continue;
+          try {
+            const dbUser = await UserModel.findOne(lookup);
+            if (dbUser) {
+              dbRoleId = dbUser.role_id;
+              debug.db_lookup = lookup;
+              debug.db_role_id = dbRoleId;
+              debug.db_role_id_type = typeof dbRoleId;
+              debug.db_email = dbUser.email;
+              if (Number(dbRoleId) === 1) { adminByDb = true; break; }
+            } else {
+              debug.db_lookup_failed = lookup;
+            }
+          } catch (e) {
+            debug.db_error = String(e && e.message || e);
+          }
+        }
+      } else {
+        debug.no_user_model = true;
+      }
+
+      // (B) Fallback: klappt auch, wenn getRootState/User-Modell im Plugin
+      //     nicht auffindbar ist — direkt aus dem Session-Objekt lesen und
+      //     Number/String tolerieren.
+      let adminBySession = false;
+      for (const cand of uCandidates) {
+        if (!cand) continue;
+        const rid = cand.role_id !== undefined ? cand.role_id : cand.roleId;
+        if (rid !== undefined && (Number(rid) === 1 || String(rid) === "1")) {
+          adminBySession = true;
+          break;
+        }
+      }
+
+      // (C) Referer-Fallback: die Plugin-Config-Seite ist mit Saltcorns
+      //     isAdmin-Middleware geschützt (packages/server/routes/plugins.ts).
+      //     Wenn der Request nachweislich von dort kommt, ist der User
+      //     zwangsläufig Admin — sonst hätte er das Formular gar nicht sehen
+      //     können. Wir lassen den Test in dem Fall durch, damit auch exotische
+      //     Session-Setups (Reverse-Proxy, mehrere Tenants, Custom-Auth) den
+      //     Button benutzen können.
+      const referer = String((req && req.headers && (req.headers.referer || req.headers.referrer)) || "");
+      const adminByReferer = /\/plugins?\/(configure|saltcorn-samba)/i.test(referer);
+      debug.admin_by_referer = adminByReferer;
+
+      const isAdmin = adminByDb || adminBySession || adminByReferer;
+      debug.admin_by_db = adminByDb;
+      debug.admin_by_session = adminBySession;
+
+      if (!isAdmin) {
+        return res.status(403).json({
+          ok: false,
+          error: "Only admins can test the connection.",
+          code: "NOT_ADMIN",
+          hint:
+            "Der Server hat Sie für diese Anfrage nicht als Administrator erkannt. " +
+            "Sehen Sie im Panel „Session-Diagnose“ unten, welche Werte Saltcorn dem " +
+            "Plugin übergibt — und melden Sie sich ggf. neu an. Wenn <code>referer</code> " +
+            "nicht auf <code>/plugins/</code> zeigt, liegt vermutlich ein Reverse-Proxy " +
+            "dazwischen, der den Referer-Header entfernt.",
+          debug,
+        });
       }
       // Accept both JSON and URL-encoded bodies. express.json and
       // express.urlencoded are both installed globally by Saltcorn.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "saltcorn-samba",
-  "version": "0.3.6",
+  "version": "0.3.8",
   "description": "Saltcorn plugin: browse, upload, rename and delete files on a Samba/CIFS share. File-manager view, directory tree, inline PDF viewer, external-app open (smb://).",
   "main": "index.js",
   "scripts": {