saltcorn-samba 0.4.8 → 0.4.9

package/CHANGELOG.md

@@ -4,6 +4,36 @@ 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.9] – 2026-07-05
+
+### Fixed – **`0xC0000033` beim Auflisten des Basispfads (smb3-client-vs-Samba-4.23-Bug jetzt auch für Basispfade abgefangen)**
+
+Auch in 0.4.8 lief der Verbindungstest mit gesetztem Basispfad in
+`QUERY_DIRECTORY failed: 0xC0000033` (`OBJECT_NAME_INVALID`). Ohne
+Basispfad war die Verbindung korrekt und zeigte den gelben Hinweis.
+
+**Ursache:** `smb3-client` sendet `QUERY_DIRECTORY` mit hart kodiertem
+`FileInformationClass = 37` (`FileIdBothDirectoryInformation`). Samba
+4.23 lehnt das für bestimmte Verzeichnisse ab — auch dann, wenn der
+Ordner existiert, geöffnet und `stat`-bar ist. Genau derselbe Bug, der
+bisher nur beim Auflisten des Share-Roots als ‚Root nicht auflistbar'
+aufgefangen wurde, kann auch den konfigurierten Basispfad selbst
+treffen.
+
+**Fix:** Der Fallback greift jetzt auch für Basispfade. Wenn
+`client.readdir("")` mit `0xC0000033` scheitert, versucht die Test-Route
+zusätzlich `client.stat("")` auf denselben Pfad:
+
+- **stat OK** → Verbindung + Basispfad bestätigt, gelber Hinweis mit
+  Erklärung des bekannten Bugs und Workaround-Vorschlägen.
+- **stat schlägt auch fehl** → normaler Fehlerpfad (Basispfad-
+  Diagnose mit Schreibvarianten-Probes) läuft weiter wie bisher.
+
+Damit lassen sich Basispfade eintragen, auf die `readdir` per
+`FileIdBothDirectoryInformation` scheitert — der Download und die
+Unterordner-Ansicht funktionieren über die normalen SMB-Calls
+trotzdem.
+
 ## [0.4.8] – 2026-07-05
 
 ### Fixed – **Basispfad wurde beim Verbindungstest doppelt vorangestellt (die eigentliche Ursache)**

package/index.js

@@ -872,18 +872,30 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
             // subdirectory to test).
             const effectivePath = testCfg.share + (baseForDisplay ? "/" + baseForDisplay : "");
             const isRootProbe = !baseForDisplay;
-            const isRootEnumBug = /0xC0000033|OBJECT_NAME_INVALID|Share-Root/i.test(msg);
-            if (isRootProbe && isRootEnumBug) {
-              // Share-root enumeration is a known smb3-client-vs-Samba
-              // limitation. Fall back to stat("") which proves the whole
-              // handshake works even if we can't list the root.
+            // smb3-client hardcodes FileInformationClass=37
+            // (FileIdBothDirectoryInformation) in QUERY_DIRECTORY. Some
+            // Samba 4.23+ configurations reject that with 0xC0000033
+            // (OBJECT_NAME_INVALID) for specific directories — even though
+            // the same directory can be opened, stat'd and traversed. In
+            // that case we fall back to a plain stat: if the folder exists
+            // (stat succeeds) we can prove the connection works, but flag
+            // that enumeration is broken so the caller shows the yellow
+            // hint instead of a red error.
+            const isEnumBug = /0xC0000033|OBJECT_NAME_INVALID|QUERY_DIRECTORY failed/i.test(msg);
+            if (isEnumBug) {
               try {
                 await client.stat("");
                 const marker = [];
-                marker._rootNotEnumerable = true;
+                if (isRootProbe) {
+                  marker._rootNotEnumerable = true;
+                } else {
+                  marker._basePathNotEnumerable = true;
+                  marker._basePathForDisplay = baseForDisplay;
+                }
                 return marker;
               } catch (statErr) {
-                throw err;
+                // stat also failed — fall through to normal error handling
+                // (missing-basepath diagnostics or generic rethrow).
               }
             }
             // NAME_NOT_FOUND / PATH_NOT_FOUND on a configured base_path.
@@ -990,6 +1002,7 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
         });
         const took = Date.now() - started;
         const rootNotEnum = Array.isArray(listing) && listing._rootNotEnumerable === true;
+        const baseNotEnum = Array.isArray(listing) && listing._basePathNotEnumerable === true;
         return res.json({
           ok: true,
           server: testCfg.server,
@@ -1011,6 +1024,16 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
               "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."
+            : 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."
             : undefined,
         });
       } catch (err) {

package/package.json

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