saltcorn-samba 0.4.0 → 0.4.2

package/CHANGELOG.md

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

@@ -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",
               }),
@@ -795,10 +791,41 @@ 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) {
+            // 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 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,
@@ -814,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.0",
+  "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

@@ -273,7 +273,39 @@ 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 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 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.
       const CHUNK = 16;