saltcorn-samba 0.4.1 → 0.4.2

package/CHANGELOG.md

@@ -4,6 +4,41 @@ 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.2] – 2026-07-05
+
+### Fixed – **Dropdown-Felder zeigen `[object Object]` statt Optionen**
+
+Die neuen Felder `signing_mode` und `encryption_mode` wurden in der
+Saltcorn-UI mit `[object Object]` als einzige Auswahl gerendert.
+
+**Ursache:** In der genutzten Saltcorn-Version werden `attributes.options`
+als einfaches String-Array erwartet (`["a", "b", "c"]`), nicht als
+`{value, label}`-Objekt. Die Objekt-Form ist erst in neueren Builds
+vollständig unterstützt; sonst castet das UI die Objekte zu Strings.
+
+**Fix:** `options` auf String-Array umgestellt —
+`["if-offered", "required", "disabled"]`. Die Beschriftung bleibt in
+der `sublabel` erhalten (der Config-Wizard erklärt jeden Wert dort).
+
+### Fixed – **`STATUS_OBJECT_NAME_NOT_FOUND` durch fehlgeleiteten Root-Fallback**
+
+Der 0.4.1-Fallback `share/.` triggerte auf Samba einen anderen NT-Status:
+`STATUS_OBJECT_NAME_NOT_FOUND` (0xC0000034 / `ENOENT`), weil Samba `.` als
+literalen Dateinamen sucht statt als Current-Directory-Marker (das ist im
+POSIX-Layer, nicht im SMB2-Protokoll). Der zweite Fallback `share/*` ist
+protokoll-illegal (Wildcard im CREATE) und wird ebenfalls abgelehnt.
+
+**Neuer Ansatz:** Wir versuchen den Fallback erst gar nicht. Statt zu
+raten geben wir eine klare deutsche Fehlermeldung aus: **einen Basispfad
+setzen**. Das ist die einzige zuverlässige Lösung, solange `smb3-client`
+die `FileInformationClass` nicht konfigurierbar macht.
+
+Die Test-Route bleibt weiterhin nachsichtig: bei `OBJECT_NAME_INVALID`
+auf dem Root fällt sie auf `stat("")` zurück und meldet ein Erfolg mit
+Hinweis („Verbindung + Anmeldung erfolgreich, aber Share-Root nicht
+direkt auflistbar — bitte Basispfad setzen“). Der Hinweis wird als gelbe
+Box im Test-Ergebnis angezeigt.
+
 ## [0.4.1] – 2026-07-05
 
 ### Fixed – **`QUERY_DIRECTORY failed: 0xC0000033` auf Share-Root (Samba 4.20+/4.23+)**

package/index.js

@@ -160,6 +160,9 @@ window.sambaTestConn = async function(btn) {
       var rows = (data.entries||[]).map(function(e){
         return '<li>'+ (e.isDirectory?'📁 ':'📄 ') + String(e.name).replace(/[<>&]/g,'?') +'</li>';
       }).join('');
+      var noteHtml = data.note
+        ? '<div style="margin-top:.5rem;padding:.4rem .6rem;background:#fff3cd;border:1px solid #ffeeba;border-radius:.25rem"><b>Hinweis:</b> ' + String(data.note).replace(/[<>&]/g,'?') + '</div>'
+        : '';
       out.innerHTML =
         '<div class="alert alert-success">' +
           '<b>✓ Verbindung erfolgreich</b> (' + data.duration_ms + ' ms)<br>' +
@@ -167,6 +170,7 @@ window.sambaTestConn = async function(btn) {
           'Basispfad: <code>' + data.base_path + '</code>, Benutzer: <code>' + data.username + '</code><br>' +
           'Einträge gefunden: <b>' + data.entry_count + '</b>' + (data.truncated ? ' (erste 20 unten)' : '') +
           (rows ? '<ul style="margin:.5rem 0 0 1rem">' + rows + '</ul>' : '') +
+          noteHtml +
         '</div>';
     } else {
       var a = data && data.attempted || {};
@@ -301,11 +305,7 @@ const configuration_workflow = () =>
                 type: "String",
                 required: true,
                 attributes: {
-                  options: [
-                    { value: "if-offered", label: "if-offered (Standard)" },
-                    { value: "required",   label: "required (strikt)" },
-                    { value: "disabled",   label: "disabled (aus)" },
-                  ],
+                  options: ["if-offered", "required", "disabled"],
                 },
                 default: "if-offered",
               }),
@@ -321,11 +321,7 @@ const configuration_workflow = () =>
                 type: "String",
                 required: true,
                 attributes: {
-                  options: [
-                    { value: "if-offered", label: "if-offered (Standard)" },
-                    { value: "required",   label: "required (strikt)" },
-                    { value: "disabled",   label: "disabled (aus)" },
-                  ],
+                  options: ["if-offered", "required", "disabled"],
                 },
                 default: "if-offered",
               }),
@@ -804,18 +800,32 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
           try {
             return await client.readdir(rel);
           } catch (err) {
-            const msg = String((err && err.message) || err || "");
+            // Look at both the wrapper error and its underlying cause.
+            const causeMsg = String((err && err.cause && err.cause.message) || "");
+            const msg      = String((err && err.message) || err || "") + " " + causeMsg;
             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 [];
+            const isRootEnumBug = /0xC0000033|OBJECT_NAME_INVALID|Share-Root/i.test(msg);
+            if (isRootProbe && isRootEnumBug) {
+              // Fall back: proof-of-life via stat on share root. This
+              // confirms TCP + Negotiate + Session + Auth + TREE_CONNECT
+              // without hitting the broken QUERY_DIRECTORY path.
+              try {
+                await client.stat("");
+                // Signal to the caller that the connection works, but
+                // the share root cannot be enumerated on this server.
+                const marker = [];
+                marker._rootNotEnumerable = true;
+                return marker;
+              } catch (statErr) {
+                // stat also failed — surface the original error.
+                throw err;
+              }
             }
             throw err;
           }
         });
         const took = Date.now() - started;
+        const rootNotEnum = Array.isArray(listing) && listing._rootNotEnumerable === true;
         return res.json({
           ok: true,
           server: testCfg.server,
@@ -831,6 +841,13 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
             isDirectory: !!(e && (e.isDirectory === true || (e.stats && e.stats.isDirectory && e.stats.isDirectory()))),
           })),
           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."
+            : undefined,
         });
       } catch (err) {
         // Turn opaque SMB / socket errors into actionable hints.

package/package.json

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

@@ -279,36 +279,32 @@ async function buildClient(config) {
       } 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 || "");
+        // 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 || "");
         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;
+        const isRootEnumBug =
+          /0xC0000033|OBJECT_NAME_INVALID|QUERY_DIRECTORY/i.test(msg);
+        if (isRoot && isRootEnumBug) {
+          const e = new Error(
+            "Das Share-Root lässt sich auf diesem Samba-Server nicht " +
+            "direkt auflisten (" + (msg || "QUERY_DIRECTORY failed") + "). " +
+            "Bitte in der Plugin-Config einen Basispfad setzen (z.\u202fB. " +
+            "einen Unterordner der Freigabe wie „daten“ oder „projekte“) — " +
+            "dann funktionieren alle Directory-Listings innerhalb dieses " +
+            "Unterordners. Details siehe README, Abschnitt „Troubleshooting → " +
+            "QUERY_DIRECTORY auf Share-Root“."
+          );
+          e.cause = err;
+          throw e;
         }
+        throw err;
       }
       // Parallel enrichment. Bounded to a reasonable concurrency to avoid
       // saturating the SMB session on huge directories.