saltcorn-samba 0.4.13 → 0.4.14

package/CHANGELOG.md

@@ -4,6 +4,50 @@ 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.4.14] – 2026-07-06
+
+### Changed – Code-Review / Aufräumen (keine Verhaltensänderung)
+
+Interne QS-Runde nach dem v0.4.13-Fix. Der Bug war behoben, aber die
+Kommentare, Fehlertexte und die Fallback-Logik im Connection-Test
+verwiesen noch auf die alte, mittlerweile widerlegte Hypothese
+(`FileInformationClass=37 hart kodiert`). Alle diese Stellen wurden
+auf den tatsächlichen Root Cause aktualisiert bzw. gelöscht.
+
+**readdir-compat.js:**
+- Docstring priorisiert den tatsächlichen Hauptbug (leeres Pattern
+  auf Folge-Pages) und schiebt den FileNameOffset-Bug in die Sekundär-
+  rolle (defensiv abgefangen, wird bei aktueller Loop nie getriggert).
+- Encoder-Docstring erklärt, dass der `pat.length === 0`-Zweig nur
+  noch defensive Sicherheit ist.
+
+**smb-client.js:**
+- Der lange, spekulative Kommentar-Block über „Some Samba builds
+  reject QUERY_DIRECTORY on the share root because FileInformation-
+  Class=37 is hardcoded“ wurde entfernt. Der wahre Grund steht jetzt
+  kurz und präzise dort.
+- Die 0xC0000033-Fehlermeldung verweist nicht mehr auf v0.4.12 als Fix
+  und nicht mehr auf `tools/diag-basepath.js`, sondern auf
+  `tools/diag-wire.js` (das aussagekräftigere Tool).
+
+**index.js (Connection-Test-Route):**
+- Der „gelber Hinweis“-Fallback bleibt als Safety Net, wird aber nur
+  noch aktiv, wenn `readdir` fehlschlägt und `stat` erfolgreich ist —
+  ein Zustand, der seit v0.4.13 nicht mehr auftreten sollte.
+- Der Hinweistext sagt jetzt „unerwarteter Zustand, bitte diag-wire.js
+  ausführen“ statt der alten, jetzt falschen Erklärung mit
+  `FileInformationClass=37`.
+
+**tools/diag-basepath.js:**
+- FIX: `client.disconnect()` → `client.close()`. smb3-client's Client
+  hat `close()`, nicht `disconnect()` (das war die marsaud-API). Der
+  Aufruf schlug bisher am Ende jedes Diagnostic-Runs mit einer
+  TypeError-Meldung fehl, ohne aber das Resultat zu beeinträchtigen.
+
+**tools/diag-wire.js:**
+- FIX: Zusätzlich zu `open.close()` wird jetzt auch `client.close()`
+  aufgerufen, damit der TCP-Socket sauber geschlossen wird.
+
 ## [0.4.13] – 2026-07-06
 
 ### Fixed – **QUERY_DIRECTORY leere Patterns → 0xC0000033 (die eigentliche Ursache)**

package/index.js

@@ -1019,21 +1019,18 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
           })),
           truncated: Array.isArray(listing) && listing.length > 20,
           note: rootNotEnum
-            ? "Verbindung + Anmeldung erfolgreich. Der Server erlaubt jedoch " +
-              "kein direktes Auflisten des Share-Roots (bekanntes Samba-" +
-              "Verhalten mit smb3-client). Bitte setzen Sie einen Basispfad " +
-              "in der Plugin-Config (z.\u202fB. einen Unterordner der " +
-              "Freigabe) — dann funktioniert der File-Manager vollständig."
+            ? "Verbindung + Anmeldung erfolgreich, aber der Share-Root konnte " +
+              "nicht aufgelistet werden (stat OK, readdir schlug fehl). Das ist " +
+              "ein unerwarteter Zustand \u2014 seit v0.4.13 sollte readdir gegen " +
+              "Samba fehlerfrei laufen. Bitte tools/diag-wire.js ausf\u00fchren " +
+              "und den Report melden."
             : baseNotEnum
             ? "Verbindung + Anmeldung erfolgreich, Basispfad \u201e" +
               (listing._basePathForDisplay || testCfg.base_path || "") +
-              "\u201c wurde vom Server best\u00e4tigt (stat OK). Das direkte " +
-              "Auflisten dieses Ordners lehnt Samba 4.23 mit smb3-client " +
-              "aktuell ab (bekannter Bug: FileInformationClass=37 hart " +
-              "kodiert, siehe README). Ordner-Unterb\u00e4ume, Datei-Downloads " +
-              "und PDF-Ansicht funktionieren \u2014 nur die Wurzel des " +
-              "Basispfads kann nicht direkt aufgelistet werden. Workaround: " +
-              "tieferen Basispfad setzen oder Unterordner ansprechen."
+              "\u201c wurde vom Server best\u00e4tigt (stat OK), das direkte " +
+              "Auflisten schlug aber fehl. Das ist ein unerwarteter Zustand \u2014 " +
+              "seit v0.4.13 sollte readdir gegen Samba fehlerfrei laufen. Bitte " +
+              "tools/diag-wire.js ausf\u00fchren und den Report melden."
             : undefined,
         });
       } catch (err) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "saltcorn-samba",
-  "version": "0.4.13",
+  "version": "0.4.14",
   "description": "Saltcorn plugin: browse, upload, rename and delete files on a Samba/CIFS share via SMB 3.1.1 (AES-CMAC signing, optional encryption). File-manager view, directory tree, inline PDF viewer, external-app open (smb://).",
   "main": "index.js",
   "scripts": {

package/readdir-compat.js

@@ -1,24 +1,30 @@
 "use strict";
 
 /**
- * readdir-compat.js  —  saltcorn-samba 0.4.13
+ * readdir-compat.js  —  saltcorn-samba 0.4.14
  *
  * Behebt zwei QUERY_DIRECTORY-Wire-Bugs in smb3-client@0.2.0 die
  * gegen Samba 4.23 zu STATUS_OBJECT_NAME_INVALID (0xC0000033) führen.
  *
- * BUG 1 (FileNameOffset bei leerem Pattern):
+ * BUG 1 (leeres Pattern auf Folge-Pages)  ─ Hauptursache des 0xC0000033:
+ *   smb3-client's `readdirAll` sendet ab der 2. Enumeration-Page
+ *   `searchPattern=""`. Windows toleriert das, Samba 4.23 lehnt es
+ *   strikt ab: `source3/smbd/smb2_query_directory.c` prüft
+ *   `in_file_name[0] == '\0'` und antwortet mit
+ *   STATUS_OBJECT_NAME_INVALID.
+ *   Fix (in readdirAllFixed): Auf jeder Page `*` senden. RESTART_SCANS
+ *   nur beim ersten Request. Samba beendet nach der letzten Seite
+ *   sauber mit STATUS_NO_MORE_FILES.
+ *
+ * BUG 2 (FileNameOffset bei leerem Pattern) ─ defensiv abgefangen:
  *   smb3-client's `encodeQueryDirectoryRequest` setzt `FileNameOffset`
  *   immer auf 96 — auch wenn `FileNameLength === 0` gesendet wird.
  *   MS-SMB2 §2.2.33 verlangt für diesen Fall FileNameOffset = 0.
- *
- * BUG 2 (Leeres Pattern auf Folge-Pages):  ← Der Hauptbug!
- *   smb3-client sendet ab der 2. Enumeration-Page `searchPattern=""`.
- *   Windows toleriert das, Samba 4.23 lehnt es strikt ab. Belegt via
- *   diag-wire.js: `smb2_query_directory.c` prüft `in_file_name[0] ==
- *   '\0'` und antwortet dann mit STATUS_OBJECT_NAME_INVALID.
- *   Fix: Auf jeder Page `*` senden. RESTART_SCANS nur beim ersten Mal.
- *   Samba beendet die Enumeration dann korrekt mit
- *   STATUS_NO_MORE_FILES auf einem Folge-Request ohne RESTART_SCANS.
+ *   Da wir seit Bug-1-Fix ohnehin nie mit leerem Pattern senden, kann
+ *   dieser Bug nicht mehr getriggert werden — encodeQueryDirectory-
+ *   RequestFixed behandelt den Fall trotzdem spec-konform, falls ein
+ *   Aufrufer den Encoder direkt benutzt oder ein zukünftiger Server
+ *   noch strenger prüft.
  *
  * Strategie:
  *   Statt smb3-client's `client.readdir()` / `readdirAll` verwenden wir
@@ -26,8 +32,9 @@
  *   verwendet (Open, Close, Tree-Connect bleiben unverändert) und nur
  *   die kaputte QUERY_DIRECTORY-Loop durch eine spec-konforme ersetzt.
  *
- * Nur die interne wire-Struktur wird via file:// URL importiert, weil
- * smb3-client's `exports`-Gate keine Subpath-Imports zulässt.
+ * Die internen wire-Module werden via `file://` URL geladen, weil
+ * smb3-client's `exports`-Gate keine Subpath-Imports über den
+ * Package-Namen zulässt (nicht einmal `smb3-client/package.json`).
  */
 
 const fs = require("fs");
@@ -134,9 +141,15 @@ function loadInternals() {
 
 /**
  * Spec-konformer Encoder für SMB2 QUERY_DIRECTORY (MS-SMB2 §2.2.33).
- * Anders als das Original:
- *   - FileNameOffset = 0, wenn kein Suchpattern (statt 96)
- * Ansonsten identisch (Struktur + Padding-Byte).
+ *
+ * Unterschied zum Original in smb3-client@0.2.0:
+ *   `FileNameOffset` wird auf 0 gesetzt, wenn `pat.length === 0`
+ *   (statt hart auf 64+32=96). MS-SMB2 §2.2.33 verlangt bei
+ *   `FileNameLength=0` genau `FileNameOffset=0`.
+ *
+ * In der aktuellen Compat-Loop wird immer `pat="*"` gesendet, daher
+ * wird dieser Zweig nie erreicht. Er bleibt als defensive Absicherung
+ * gegen Aufrufer, die den Encoder direkt mit leerem Pattern nutzen.
  */
 function encodeQueryDirectoryRequestFixed(I, req) {
   const pat = req.searchPattern

package/smb-client.js

@@ -227,8 +227,8 @@ async function buildClient(config) {
    * Convert a readdirCompat rich-Dirent (which already carries name,
    * size, mtime, ctime, and isDirectory()) directly to the legacy
    * `@marsaud/smb2`-compatible shape the rest of the plugin expects.
-   * No extra stat roundtrip needed — v0.4.12 gets attributes straight
-   * from the QUERY_DIRECTORY response.
+   * No extra stat roundtrip needed — attributes come straight from the
+   * single QUERY_DIRECTORY response.
    */
   function mapDirent(dirent) {
     return {
@@ -255,42 +255,33 @@ async function buildClient(config) {
      */
     async readdir(rel) {
       const full = resolvePath(rel);
-      // v0.4.12: readdirCompat ist der ALLEINIGE readdir-Pfad. Grund:
-      // smb3-client@0.2.0 hat einen Wire-Format-Bug in
-      // encodeQueryDirectoryRequest, der gegen Samba 4.23 zu
-      // STATUS_OBJECT_NAME_INVALID (0xC0000033) führt. readdirCompat
-      // implementiert QUERY_DIRECTORY MS-SMB2-§2.2.33-konform und
-      // umgeht damit den Bug. Es gibt keinen Fallback auf
-      // client.readdir(), weil das den bekannten Bug hervorruft.
+      // readdirCompat ist der ALLEINIGE readdir-Pfad. Grund: smb3-
+      // client@0.2.0 hat zwei QUERY_DIRECTORY-Wire-Bugs, die gegen
+      // Samba 4.23 zu STATUS_OBJECT_NAME_INVALID (0xC0000033) führen
+      // (leeres Pattern auf Folge-Pages + falscher FileNameOffset).
+      // readdirCompat implementiert QUERY_DIRECTORY MS-SMB2-§2.2.33-
+      // konform und umgeht beide Bugs. Kein Fallback auf
+      // client.readdir() — das würde die Bugs reproduzieren.
       let dirents;
       try {
         dirents = await readdirCompat(client, full, { rich: true });
       } catch (errCompat) {
         const err = errCompat;
-        // Some Samba builds (observed on 4.20+/4.23+) reject
-        // SMB2 QUERY_DIRECTORY on the *share root* with
-        // STATUS_OBJECT_NAME_INVALID (0xC0000033) because smb3-client
-        // asks for FileIdBothDirectoryInformation (class 37) on a
-        // handle opened with an empty filename. We cannot swap the
-        // information class from userland, and Samba refuses both
-        // "." (→ STATUS_OBJECT_NAME_NOT_FOUND) and "*" (→ CREATE with
-        // wildcard is protocol-illegal). The clean way out is:
-        // require the caller to configure a real base_path so every
-        // readdir happens inside a directory the server can enumerate.
         const msg = String((err && err.message) || err || "");
         // Falls der Compat-Pfad selbst mit 0xC0000033 zurückkommt,
         // liegt entweder ein weiterer, unbekannter Wire-Bug vor oder
-        // Samba lehnt aus einem anderen Grund ab. Wir geben eine
-        // gezielte Meldung mit Hinweis auf den Bug-Report.
+        // Samba lehnt aus einem anderen Grund ab (z. B. fehlende
+        // Rechte auf die Zielressource). Wir geben eine gezielte
+        // Meldung mit Handlungsanweisung.
         if (/0xC0000033|OBJECT_NAME_INVALID/i.test(msg)) {
           const e = new Error(
-            "Samba lehnt QUERY_DIRECTORY auf diesem Pfad ab " +
-              "(STATUS_OBJECT_NAME_INVALID, 0xC0000033). " +
-              "Der bekannte smb3-client-Wire-Bug wurde in v0.4.12 gefixt — " +
-              "falls dieser Fehler weiterhin auftritt, liegt ein zweiter, " +
-              "noch unbekannter Wire-Format-Fehler vor. Bitte " +
-              "tools/diag-basepath.js ausführen und Report zurückschicken. " +
-              "Details: " +
+            "Samba lehnt QUERY_DIRECTORY auf diesem Pfad unerwartet ab " +
+              "(STATUS_OBJECT_NAME_INVALID, 0xC0000033). Die bekannten " +
+              "smb3-client-Wire-Bugs sind in dieser Plugin-Version " +
+              "bereits gepatcht — falls dieser Fehler dennoch auftritt, " +
+              "liegt ein bislang unbekannter Wire-Format-Fehler oder ein " +
+              "Rechteproblem vor. Bitte tools/diag-wire.js ausführen " +
+              "und den Report zurücksenden. Details: " +
               msg
           );
           e.cause = err;
@@ -320,9 +311,9 @@ async function buildClient(config) {
         }
         throw err;
       }
-      // v0.4.12: readdirCompat gibt schon Rich-Dirents zurück (Name,
-      // Größe, mtime, ctime, isDirectory). Kein zusätzlicher stat-Fan-out
-      // nötig — ein einziger QUERY_DIRECTORY-Roundtrip liefert alles.
+      // readdirCompat gibt Rich-Dirents zurück (Name, Größe, mtime,
+      // ctime, isDirectory). Kein zusätzlicher stat-Fan-out nötig —
+      // ein einziger QUERY_DIRECTORY-Roundtrip liefert alles.
       return dirents.map(mapDirent);
     },
 

package/tools/diag-basepath.js

@@ -91,7 +91,10 @@ async function main() {
     step++;
   }
 
-  await client.disconnect();
+  // smb3-client's Client has close(), not disconnect() — the old
+  // marsaud API used disconnect. Use close() and ignore errors so a
+  // failing probe still tears down cleanly.
+  try { await client.close(); } catch (_) {}
   console.log("Done.");
 }
 

package/tools/diag-wire.js

@@ -325,6 +325,9 @@ async function main() {
   try {
     await open.close();
   } catch (_) {}
+  try {
+    await client.close();
+  } catch (_) {}
   console.log("\nDone.");
 }