saltcorn-samba 0.4.0 → 0.4.1

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.1] – 2026-07-05
+
+### Fixed – **`QUERY_DIRECTORY failed: 0xC0000033` auf Share-Root (Samba 4.20+/4.23+)**
+
+Symptom nach dem 0.4.0-Update: der Verbindungstest schlägt sofort fehl mit
+
+```
+Fehler: QUERY_DIRECTORY failed: 0xC0000033
+```
+
+(`STATUS_OBJECT_NAME_INVALID`). Betroffen sind Setups **ohne Basispfad** —
+sobald ein Basispfad gesetzt ist, greift der Fehler nicht.
+
+**Ursache:** `smb3-client` öffnet den Share-Root im SMB2-CREATE mit einem
+leeren Filename (`""`). Moderne Samba-Versionen (bestehen jedenfalls
+**4.20+**, bestätigt auf **4.23.9**) akzeptieren das für CREATE, weisen
+aber das anschließende `QUERY_DIRECTORY` mit `FileIdBothDirectoryInformation`
+auf dem leeren Namen als `OBJECT_NAME_INVALID` zurück. Vergleichbare
+Probleme sind aus anderen Java-/Go-SMB-Client-Bibliotheken bekannt (z. B.
+[smbj#80](https://github.com/hierynomus/smbj/issues/80)).
+
+**Lösung:**
+
+1. `smb-client.js#readdir("")` bekommt einen Fallback: wenn der
+   Server auf dem leeren Root mit `OBJECT_NAME_INVALID` antwortet, wird
+   die Auflistung noch einmal mit `share/.` (aktuelles Verzeichnis) und,
+   falls das ebenfalls scheitert, mit `share/*` (Wildcard) probiert.
+   Erst wenn auch das nicht klappt, wird eine deutsche Fehlermeldung mit
+   Handlungsanweisung ausgelöst.
+2. Die `/sambatest`-Route greift auf `client.stat("")` zurück, wenn das
+   Root-`readdir` mit `OBJECT_NAME_INVALID` scheitert. `stat("")`
+   verwendet CREATE ohne `DIRECTORY_FILE`-Flag und ohne QUERY_DIRECTORY —
+   das beweist Netzwerk + Negotiate + Session-Setup + TREE_CONNECT + Auth
+   ohne die problematische Query. Der Test liefert dann `entry_count: 0`
+   und den Hinweis, dass ein Basispfad gesetzt werden sollte, sofern der
+   Server das Root-Listing nicht anders bereitstellt.
+
+### Empfehlung
+
+Wenn Ihr Samba-Server das Share-Root-`QUERY_DIRECTORY` weiterhin ablehnt,
+setzen Sie in der Plugin-Config einen **Basispfad** (z. B. `daten` oder
+`projekte`) — dann sind alle Directory-Listings innerhalb dieses
+Unterverzeichnisses, was durchgängig funktioniert.
+
 ## [0.4.0] – 2026-07-05
 
 ### ⚠️ BREAKING CHANGES

package/index.js

@@ -795,8 +795,25 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
       try {
         const listing = await withClient(testCfg, async (client) => {
           // If base_path is set, list it — that also verifies traversal.
+          // For the share-root case we first try readdir; some Samba builds
+          // reject QUERY_DIRECTORY on the empty root name, so we fall back
+          // to stat("") — that already proves the whole handshake
+          // (TCP + Negotiate + Session + TREE_CONNECT + Auth) works, which
+          // is all the connection test actually promises.
           const rel = testCfg.base_path ? sanitizeRelativePath(testCfg.base_path) : "";
-          return await client.readdir(rel);
+          try {
+            return await client.readdir(rel);
+          } catch (err) {
+            const msg = String((err && err.message) || err || "");
+            const isRootProbe = !rel;
+            const isNameInvalid = /0xC0000033|OBJECT_NAME_INVALID/i.test(msg);
+            if (isRootProbe && isNameInvalid) {
+              // Fall back: proof-of-life via stat on share root.
+              await client.stat("");
+              return [];
+            }
+            throw err;
+          }
         });
         const took = Date.now() - started;
         return res.json({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "saltcorn-samba",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "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/smb-client.js

@@ -273,7 +273,43 @@ async function buildClient(config) {
      */
     async readdir(rel) {
       const full = resolvePath(rel);
-      const dirents = await client.readdir(full, { withFileTypes: true });
+      let dirents;
+      try {
+        dirents = await client.readdir(full, { withFileTypes: true });
+      } catch (err) {
+        // Some Samba builds (observed on 4.20+/4.23+) reject
+        // SMB2 QUERY_DIRECTORY on the *share root* with
+        // STATUS_OBJECT_NAME_INVALID (0xC0000033) because the CREATE
+        // used an empty filename. The fallback below re-issues the
+        // request with an explicit path (".") which most Samba VFS
+        // modules accept for the current directory.
+        const status = (err && (err.status || err.code)) || "";
+        const msg    = String((err && err.message) || err || "");
+        const isRoot = !rel && !basePath;
+        const isNameInvalid =
+          /0xC0000033|STATUS_OBJECT_NAME_INVALID|OBJECT_NAME_INVALID/i.test(msg) ||
+          status === 0xc0000033;
+        if (isRoot && isNameInvalid) {
+          try {
+            dirents = await client.readdir(shareName + "/.", { withFileTypes: true });
+          } catch (_) {
+            // Second fallback: an explicit wildcard segment.
+            try {
+              dirents = await client.readdir(shareName + "/*", { withFileTypes: true });
+            } catch (_) {
+              const e = new Error(
+                "QUERY_DIRECTORY auf dem Share-Root schlug fehl (" +
+                (msg || "OBJECT_NAME_INVALID") + "). " +
+                "Setzen Sie einen Basispfad im Plugin-Konfig oder prüfen Sie die Share-Definition auf dem Server."
+              );
+              e.cause = err;
+              throw e;
+            }
+          }
+        } else {
+          throw err;
+        }
+      }
       // Parallel enrichment. Bounded to a reasonable concurrency to avoid
       // saturating the SMB session on huge directories.
       const CHUNK = 16;