saltcorn-samba 0.3.8 → 0.3.10

package/CHANGELOG.md

@@ -4,6 +4,110 @@ 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.10] – 2026-07-05
+
+### Fixed – **DNS-Fehler: `getaddrinfo ENOTFOUND "host:445"`**
+
+Nachdem 0.3.9 den Callback-Signatur-Bug behoben hatte und der Test-Button
+endlich echt gegen den SMB-Server lief, kam bei der Verbindung ein neuer
+Fehler zum Vorschein:
+
+```
+getaddrinfo ENOTFOUND 192.168.110.10:445 (ENOTFOUND)
+```
+
+**Ursache:** In `smb-client.js` wurde der UNC-Share-String so gebaut:
+
+```js
+const shareStr = `\\\\${server}${port ? ":" + port : ""}\\${share}`;
+// => "\\\\192.168.110.10:445\\buero"
+```
+
+`@marsaud/smb2` interpretiert alles zwischen den führenden `\\` und dem
+nächsten `\` als **Hostnamen** und reicht das direkt an
+`net.connect()` / `dns.lookup()` weiter. Node versucht dann
+`"192.168.110.10:445"` als Hostnamen aufzulösen — was natürlich
+fehlschlägt.
+
+**Fix:** Host und Port sauber trennen.
+
+- Der `share`-UNC-Pfad enthält jetzt **nur den Host**:
+  `\\192.168.110.10\buero`
+- Der Port wird über die separate `port`-Option an `SMB2` übergeben
+  (Default 445).
+- Zusätzlich tolerant: falls jemand versehentlich `host:445` ins
+  Server-Feld einträgt, wird der Port dort herausgezogen.
+- IPv6-Adressen in eckigen Klammern (`[::1]:445`) werden unterstützt.
+
+### Notes
+- Keine Konfig-Migration nötig. Wer bisher Host **ohne** Port eingetragen
+  hat, bekommt jetzt genau die gleiche Verbindung wie vorher — nur
+  funktionierend.
+- Wer aus Verzweiflung `IP:445` ins Server-Feld getippt hatte: das wird
+  jetzt automatisch aufgesplittet. Sauberer ist trotzdem: Host im
+  Server-Feld, Port im Port-Feld.
+
+## [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.10",
   "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": {

package/smb-client.js

@@ -116,13 +116,41 @@ function buildClient(config) {
   if (!share) throw new Error("Samba: share missing");
   if (/[\\/]/.test(share)) throw new Error("Samba: share must not contain slashes");
 
-  const shareStr = `\\\\${server}${port ? ":" + port : ""}\\${share}`;
+  // Host und Port sauber trennen. Der Server-String darf NICHT im share-UNC-
+  // Pfad landen, sonst versucht Node's DNS "host:port" als Hostnamen aufzulösen
+  // (getaddrinfo ENOTFOUND "1.2.3.4:445"). @marsaud/smb2 nimmt den Port über
+  // die separate `port`-Option entgegen; der share-String enthält nur den Host.
+  //
+  // Zusätzlich tolerant sein, falls jemand versehentlich "host:445" ins
+  // Server-Feld getippt hat — dann Port dort rausziehen.
+  let hostOnly = String(server).trim();
+  let portFromServer;
+  // IPv6-Adressen in eckigen Klammern zulassen: [::1]:445
+  const v6 = hostOnly.match(/^\[([^\]]+)\](?::(\d+))?$/);
+  if (v6) {
+    hostOnly = v6[1];
+    if (v6[2]) portFromServer = Number(v6[2]);
+  } else {
+    // Nur splitten, wenn genau EIN Doppelpunkt — sonst IPv6 ohne Klammern
+    const colonCount = (hostOnly.match(/:/g) || []).length;
+    if (colonCount === 1) {
+      const [h, p] = hostOnly.split(":");
+      if (h && /^\d+$/.test(p)) {
+        hostOnly = h;
+        portFromServer = Number(p);
+      }
+    }
+  }
+
+  const effectivePort = Number(port) || portFromServer || 445;
+  const shareStr = `\\\\${hostOnly}\\${share}`;
   const SMB2 = getSMB2();
   const smb = new SMB2({
     share: shareStr,
     domain: domain || "WORKGROUP",
     username: username || "guest",
     password: password || "",
+    port: effectivePort,
     autoCloseTimeout: 10000,
   });