saltcorn-samba 0.3.8 → 0.3.9

package/CHANGELOG.md

@@ -4,6 +4,67 @@ 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.9] – 2026-07-05
+
+### Fixed – **Root-Cause-Fix: Falsche Callback-Signatur in allen Routen**
+
+Die Diagnose-Ausgabe aus 0.3.8 hat gezeigt, dass in den Route-Handlern
+**alle** Felder von `req` leer waren (`has_req_user: false`,
+`has_session: false`, `referer: null` usw.). Ursache war eine falsche
+Callback-Signatur, die seit 0.1 in **allen 8 Routen** verwendet wurde.
+
+Saltcorn registriert Plugin-Routen so
+(`packages/server/plugin_routes_handler.js`, Zeilen 40–52):
+
+```js
+tenantRouter.post(url, error_catcher(route.callback));
+```
+
+und `error_catcher` ist definiert als
+(`packages/server/routes/utils.ts`, Zeile 427):
+
+```js
+const error_catcher = (fn) => (request, response, next) => {
+  ...; fn(request, response, next);
+};
+```
+
+Die Callbacks werden also mit der klassischen Express-Signatur
+`(req, res, next)` aufgerufen — **nicht** mit einem Objekt `{ req, res }`.
+
+Bisher stand in `index.js` überall:
+
+```js
+callback: async ({ req, res }) => { ... }
+```
+
+Dadurch wurde aus dem echten Express-`request`-Objekt versucht, `req.req`
+und `req.res` per Destructuring zu holen — beides `undefined`. Effekt:
+
+- `req.user`, `req.session`, `req.headers`, `req.body`, `req.query`,
+  `req.csrfToken()` waren alle unerreichbar.
+- Der Admin-Check konnte niemals erfolgreich sein.
+- Der Samba-Verbindungstest hat *nie* funktioniert.
+- Datei-Listing, Upload, Rename, Delete, Mkdir waren ebenfalls betroffen.
+
+**Fix:** Alle 8 Routen (`/sambadir`, `/sambafile`, `/sambalink`,
+`/sambatest`, `/sambaupload`, `/sambadelete`, `/sambarename`,
+`/sambamkdir`) auf die korrekte Signatur umgestellt:
+
+```js
+callback: async (req, res) => { ... }
+```
+
+Damit erhält jeder Handler das echte Express-`req`-Objekt mit
+`req.user`, `req.session`, `req.headers`, `req.body`, `req.csrfToken()`,
+usw. — und die gesamte Admin- und Berechtigungslogik aus 0.3.7/0.3.8
+greift jetzt so, wie sie gedacht war.
+
+### Notes
+- Kein Funktionsverlust, keine Konfig-Migration nötig.
+- Wer 0.3.8 installiert hat und den Test-Button nicht zum Laufen bekommen
+  hat: **Update auf 0.3.9 einspielen, Server neu starten, erneut testen.**
+
 ## [0.3.8] – 2026-07-05
 
 ### Changed

package/index.js

@@ -480,7 +480,7 @@ const routes = [
   {
     url: "/sambadir",
     method: "get",
-    callback: async ({ req, res }) => {
+    callback: async (req, res) => {
       const cfg = getConfig();
       if (!cfg.server) return jsonError(res, 500, "Samba plugin not configured");
       if (!canRead(req, cfg)) return jsonError(res, 403, "Forbidden");
@@ -528,7 +528,7 @@ const routes = [
   {
     url: "/sambafile",
     method: "get",
-    callback: async ({ req, res }) => {
+    callback: async (req, res) => {
       const cfg = getConfig();
       if (!cfg.server) return res.status(500).send("Samba plugin not configured");
       if (!canRead(req, cfg)) return res.status(403).send("Forbidden");
@@ -561,7 +561,7 @@ const routes = [
   {
     url: "/sambalink",
     method: "get",
-    callback: async ({ req, res }) => {
+    callback: async (req, res) => {
       const cfg = getConfig();
       if (!cfg.server) return res.status(500).send("Samba plugin not configured");
       if (!canRead(req, cfg)) return res.status(403).send("Forbidden");
@@ -617,7 +617,7 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
     // when the config-page CSRF token has already been consumed by the
     // save-workflow or when a stricter CSRF policy is applied.
     noCsrf: true,
-    callback: async ({ req, res }) => {
+    callback: async (req, res) => {
       // -----------------------------------------------------------------
       // Admin gate — komplett neu in 0.3.8:
       //
@@ -822,7 +822,7 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
   {
     url: "/sambaupload",
     method: "post",
-    callback: async ({ req, res }) => {
+    callback: async (req, res) => {
       const cfg = getConfig();
       if (!cfg.server) return jsonError(res, 500, "Samba plugin not configured");
       if (!canWrite(req, cfg)) return jsonError(res, 403, "Forbidden");
@@ -893,7 +893,7 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
   {
     url: "/sambadelete",
     method: "post",
-    callback: async ({ req, res }) => {
+    callback: async (req, res) => {
       const cfg = getConfig();
       if (!cfg.server) return jsonError(res, 500, "Samba plugin not configured");
       if (!canWrite(req, cfg)) return jsonError(res, 403, "Forbidden");
@@ -924,7 +924,7 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
   {
     url: "/sambarename",
     method: "post",
-    callback: async ({ req, res }) => {
+    callback: async (req, res) => {
       const cfg = getConfig();
       if (!cfg.server) return jsonError(res, 500, "Samba plugin not configured");
       if (!canWrite(req, cfg)) return jsonError(res, 403, "Forbidden");
@@ -987,7 +987,7 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
   {
     url: "/sambamkdir",
     method: "post",
-    callback: async ({ req, res }) => {
+    callback: async (req, res) => {
       const cfg = getConfig();
       if (!cfg.server) return jsonError(res, 500, "Samba plugin not configured");
       if (!canWrite(req, cfg)) return jsonError(res, 403, "Forbidden");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "saltcorn-samba",
-  "version": "0.3.8",
+  "version": "0.3.9",
   "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": {