saltcorn-samba 0.3.9 → 0.3.10

package/CHANGELOG.md

@@ -4,6 +4,49 @@ 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**

package/package.json

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