saltcorn-samba 0.3.6 → 0.3.7

package/CHANGELOG.md

@@ -4,6 +4,28 @@ 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.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

@@ -156,20 +156,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 +382,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 +606,38 @@ 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 — be permissive about how Saltcorn represents the role:
+      //   • req.user.role_id may be a Number (1) or a String ("1") depending
+      //     on how the session was serialised.
+      //   • In some tenant/session configs req.user is populated later; we
+      //     also check req.session.passport.user as a fallback.
+      const u =
+        (req && req.user) ||
+        (req && req.session && req.session.passport && req.session.passport.user) ||
+        null;
+      const rid = u && (u.role_id !== undefined ? u.role_id : u.roleId);
+      const isAdmin = u && (Number(rid) === 1 || String(rid) === "1");
+      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. " +
+            "Mögliche Ursachen: (1) Sie sind in einem anderen Tab abgemeldet worden, " +
+            "(2) die Konfigurations-Seite wurde in einem privaten/Inkognito-Fenster " +
+            "geöffnet ohne gültige Session, (3) Saltcorn läuft hinter einem Reverse-" +
+            "Proxy, der Cookies nicht weiterreicht. Bitte im selben Fenster neu " +
+            "anmelden und die Seite neu laden.",
+          debug: {
+            has_req_user: !!(req && req.user),
+            has_session: !!(req && req.session),
+            role_id_seen: rid === undefined ? null : String(rid),
+            role_id_type: rid === undefined ? "undefined" : typeof rid,
+            email: (u && u.email) || null,
+            user_id: (u && u.id) || null,
+          },
+        });
       }
       // 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.7",
   "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": {