saltcorn-samba 0.4.13 → 0.4.15

package/CHANGELOG.md

@@ -4,6 +4,101 @@ 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.15] – 2026-07-07
+
+### Fixed – CSRF-Fehlermeldung bei Upload / Ordner anlegen / Rename / Delete
+
+Bei jedem schreibenden Klick in der File-Manager-View („Ordner anlegen“,
+„Datei hochladen“, „umbenennen“, „löschen“) erschien der Fehler
+`Invalid CSRF token`, obwohl der Client den Token korrekt im Header
+`X-CSRF-Token` mitgeschickt hat.
+
+**Root Cause:** Saltcorns globaler `csurf`-Middleware validiert bereits
+alle POST-Routen der Plugins gegen das Session-Secret. Unser Plugin hatte
+zusätzlich eine manuelle `checkCsrf(req)`-Funktion, die den vom Client
+geschickten Token mit `req.csrfToken()` verglich. `req.csrfToken()` gibt
+aber bei jedem Aufruf einen **frisch gesalzenen** Token zurück – der
+Stringvergleich `provided !== req.csrfToken()` schlug daher immer fehl,
+auch bei völlig gültigen Requests.
+
+**Fix:** `checkCsrf` ist jetzt ein dokumentierter No-Op. Die eigentliche
+CSRF-Prüfung übernimmt weiterhin Saltcorns Middleware (kein Sicherheits-
+abbau). Betroffene Write-Routen (Upload, Mkdir, Rename, Delete) in
+`index.js` sind unverändert und funktionieren jetzt korrekt.
+
+### Added – View-Basispfad (`view_base_path`) für File-Manager und Tree
+
+Beide Views (SambaFileManager und SambaTree) haben ein neues optionales
+Konfigurationsfeld **„View-Basispfad (relativ zum Plugin-Basispfad)“**.
+Damit lässt sich der Pfad zweistufig konfigurieren:
+
+1. **Plugin-Basispfad** (weiterhin Pflicht) – z. B. `static` – legt fest,
+   in welchem Unterverzeichnis der Freigabe der Plugin-Root liegt.
+2. **View-Basispfad** (neu, optional) – z. B. `projekte/2026` – ein
+   statischer Präfix, der nur für diese eine View gilt.
+
+Beispiel: Plugin-Basispfad = `static`, View-Basispfad = `projekte/2026`
+→ die View listet den effektiven Pfad `static/projekte/2026`.
+
+Der bisherige `extra_subpath` in beiden Views ist jetzt konsistent auf
+den `from_field`-Modus beschränkt (dort weiterhin: Suffix hinter dem
+Feldwert der aktuellen Zeile). Im `static`-Modus zählt nur
+`view_base_path`.
+
+Der zusammengesetzte Pfad wird pro Segment slash-getrimmt und
+durchläuft `sanitizeRelativePath`, so dass `..`-Traversal, absolute
+Pfade, UNC-Präfixe usw. weiterhin abgelehnt werden.
+
+### Tests
+
+`test/sanitize.test.js` deckt jetzt zusätzlich die Pfad-Komposition
+(`view_base_path` + Feldwert + `extra_subpath`) inklusive Slash-
+Normalisierung, Modus-Regeln und Traversal-Ablehnung ab.
+
+## [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/README.md

@@ -194,9 +194,10 @@ Optionen:
 
 | Option | Zweck |
 |---|---|
-| **Root directory mode** | `static` = immer Base-Path aus Plugin-Config, `from_field` = Sub-Pfad kommt aus einem DB-Feld der aktuellen Zeile |
-| **Row field with sub-path** | Feld der Tabelle mit dem relativen Ordnernamen (bei `from_field`) |
-| **Extra sub-path** | statischer Suffix (z. B. `invoices`) |
+| **View-Basispfad (relativ zum Plugin-Basispfad)** | Optionaler statischer Präfix nur für diese View. Beispiel: Plugin-Basispfad = `static`, View-Basispfad = `projekte/2026` → View listet `static/projekte/2026`. Zweistufige Konfiguration: einmal serverweit im Plugin, einmal pro View. |
+| **Row-Modus** | `static` = nur der View-Basispfad zählt, `from_field` = zusätzlich wird ein DB-Feldwert der aktuellen Zeile angehängt (z. B. in einer Show-View) |
+| **Feld mit Unterpfad** | Feld der Tabelle mit dem relativen Ordnernamen (nur bei `from_field`) |
+| **Zusätzlicher Suffix (nach dem Feldwert)** | statischer Suffix, z. B. `invoices` (nur bei `from_field`) |
 | **Show hidden files** | `.dotfiles` einblenden |
 | **Allow navigating up** | Up-Button aktivieren (nur bis zum Root, kein Ausbruch) |
 | **Open PDFs / images inline** | Klick öffnet die Datei im integrierten Viewer |
@@ -204,6 +205,10 @@ Optionen:
 | **Page size** | Einträge pro Seite (0 = alle) |
 | **Panel title** | Text im Karten-Header |
 
+Der effektive Pfad wird zusammengesetzt aus (jeweils falls gesetzt):
+`Plugin-Basispfad / View-Basispfad / Feldwert / Zusätzlicher Suffix`.
+`..`-Traversal, absolute Pfade und UNC-Präfixe werden abgelehnt.
+
 Beispiel-Datenmodell für kunden-spezifische Ordner:
 
 ```
@@ -212,12 +217,13 @@ Felder:   name (String), akte_dir (String, z. B. "kunde_42/2026")
 ```
 
 Dann eine Show-View von `kunden` bauen und die `SambaFileManager`-View mit
-Mode = `from_field`, Row field = `akte_dir` einbetten.
+Row-Modus = `from_field`, Feld = `akte_dir` einbetten.
 
 ### View: `SambaTree`
 
 Kompaktere Alternative – lazy-loading Baum, ideal in einer Sidebar oder
-neben Formularen. Gleiche Path-Modi wie beim File-Manager.
+neben Formularen. Gleiche Path-Optionen wie beim File-Manager (inkl.
+**View-Basispfad**, **Row-Modus**, **Feld**, **Zusätzlicher Suffix**).
 
 ### Fieldview `samba_pdf` (deaktiviert seit 0.3.2)
 

package/filemanager-view.js

@@ -44,31 +44,42 @@ const configuration_workflow = () =>
             .map((f) => f.name);
           return new Form({
             fields: [
+              new Field({
+                name: "view_base_path",
+                label: "View-Basispfad (relativ zum Plugin-Basispfad)",
+                type: "String",
+                sublabel:
+                  "Optional. Statischer Pfad relativ zum Plugin-Basispfad, der f\u00fcr diese View gilt. " +
+                  "Beispiel: Plugin-Basispfad = 'static', View-Basispfad = 'projekte/2026' \u2192 View listet 'static/projekte/2026'. " +
+                  "Keine f\u00fchrenden/abschlie\u00dfenden Slashes n\u00f6tig. Traversal (\u201e..\u201c) wird abgelehnt.",
+              }),
               new Field({
                 name: "mode",
-                label: "Root directory mode",
+                label: "Row-Modus",
                 type: "String",
                 required: true,
                 attributes: {
                   options: ["static", "from_field"],
                 },
                 sublabel:
-                  "static = always the plugin base_path. from_field = append a row field value to base_path (only when the view is embedded in a Show view).",
+                  "static = immer nur der View-Basispfad. from_field = h\u00e4nge einen Feldwert der aktuellen Zeile an " +
+                  "(nur sinnvoll, wenn die View in einer Show-View eingebettet ist).",
                 default: "static",
               }),
               new Field({
                 name: "path_field",
-                label: "Row field with sub-path",
+                label: "Feld mit Unterpfad",
                 type: "String",
                 attributes: { options: stringFieldOptions },
                 showIf: { mode: "from_field" },
               }),
               new Field({
                 name: "extra_subpath",
-                label: "Extra sub-path (appended)",
+                label: "Zus\u00e4tzlicher Suffix (nach dem Feldwert)",
                 type: "String",
                 sublabel:
-                  "Optional. Static suffix appended after the field value. Example: 'invoices'",
+                  "Optional. Statischer Suffix, der nach dem Feldwert angeh\u00e4ngt wird. Beispiel: 'invoices'.",
+                showIf: { mode: "from_field" },
               }),
               new Field({
                 name: "show_hidden",
@@ -122,14 +133,37 @@ async function get_state_fields() {
   return [];
 }
 
+/**
+ * Compose the start path (relative to the plugin base_path) from three
+ * optional parts:
+ *   1. `view_base_path`  — static, per-view prefix
+ *   2. row field value    — only in "from_field" mode
+ *   3. `extra_subpath`   — static suffix, only in "from_field" mode
+ *
+ * Empty parts are dropped. The final joined path is passed through
+ * sanitizeRelativePath so traversal / absolute paths / backslashes are
+ * rejected consistently.
+ */
 function computeStartPath(configuration, row) {
   const parts = [];
+  if (configuration.view_base_path) {
+    parts.push(String(configuration.view_base_path));
+  }
   if (configuration.mode === "from_field" && configuration.path_field && row) {
     const v = row[configuration.path_field];
     if (v) parts.push(String(v));
   }
-  if (configuration.extra_subpath) parts.push(String(configuration.extra_subpath));
-  return sanitizeRelativePath(parts.join("/"));
+  if (
+    configuration.mode === "from_field" &&
+    configuration.extra_subpath
+  ) {
+    parts.push(String(configuration.extra_subpath));
+  }
+  const joined = parts
+    .map((p) => p.replace(/^\/+|\/+$/g, ""))
+    .filter(Boolean)
+    .join("/");
+  return sanitizeRelativePath(joined);
 }
 
 // ---------------------------------------------------------------------------

package/index.js

@@ -519,22 +519,18 @@ function jsonOk(res, extra) {
 }
 
 /**
- * Validate that the request carries a CSRF token matching the session.
- * Saltcorn injects `req.csrfToken()` when the CSRF middleware is active;
- * we accept either the `_csrf` body field or the `x-csrf-token` header.
+ * CSRF verification is handled by Saltcorn's global csurf middleware
+ * (packages/server/app.js). By the time our route handler runs, csurf
+ * has already validated req.body._csrf / X-CSRF-Token / csrf-token /
+ * xsrf-token headers against the session secret. A manual re-check
+ * here would be wrong: csrf-tokens returns a NEW salted token on
+ * every call to req.csrfToken(), so `provided !== req.csrfToken()`
+ * would fail even for perfectly valid requests.
+ *
+ * We keep this stub as the single documented spot to attach future
+ * pre-flight authorisation checks if ever needed.
  */
-function checkCsrf(req, res) {
-  if (typeof req.csrfToken !== "function") return true; // CSRF disabled globally
-  const expected = req.csrfToken();
-  const provided =
-    (req.body && req.body._csrf) ||
-    req.headers["x-csrf-token"] ||
-    req.headers["csrf-token"] ||
-    req.query._csrf;
-  if (!provided || provided !== expected) {
-    jsonError(res, 403, "Invalid CSRF token");
-    return false;
-  }
+function checkCsrf(_req, _res) {
   return true;
 }
 
@@ -1019,21 +1015,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.15",
   "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.");
 }
 

package/tree-view.js

@@ -41,30 +41,42 @@ const configuration_workflow = () =>
             .map((f) => f.name);
           return new Form({
             fields: [
+              new Field({
+                name: "view_base_path",
+                label: "View-Basispfad (relativ zum Plugin-Basispfad)",
+                type: "String",
+                sublabel:
+                  "Optional. Statischer Pfad relativ zum Plugin-Basispfad, der f\u00fcr diese View gilt. " +
+                  "Beispiel: Plugin-Basispfad = 'static', View-Basispfad = 'projekte/2026' \u2192 View listet 'static/projekte/2026'. " +
+                  "Keine f\u00fchrenden/abschlie\u00dfenden Slashes n\u00f6tig. Traversal (\u201e..\u201c) wird abgelehnt.",
+              }),
               new Field({
                 name: "mode",
-                label: "Path mode",
+                label: "Row-Modus",
                 type: "String",
                 required: true,
                 attributes: {
                   options: ["static", "from_field"],
                 },
                 sublabel:
-                  "static = always show the plugin base_path; from_field = append a row field value",
+                  "static = immer nur der View-Basispfad. from_field = h\u00e4nge einen Feldwert der aktuellen Zeile an " +
+                  "(nur sinnvoll, wenn die View in einer Show-View eingebettet ist).",
+                default: "static",
               }),
               new Field({
                 name: "path_field",
-                label: "Row field with sub-path",
+                label: "Feld mit Unterpfad",
                 type: "String",
                 attributes: { options: stringFieldOptions },
                 showIf: { mode: "from_field" },
               }),
               new Field({
                 name: "extra_subpath",
-                label: "Extra sub-path (appended)",
+                label: "Zus\u00e4tzlicher Suffix (nach dem Feldwert)",
                 type: "String",
                 sublabel:
-                  "Optional. Static suffix appended after the field value. Example: 'invoices'",
+                  "Optional. Statischer Suffix, der nach dem Feldwert angeh\u00e4ngt wird. Beispiel: 'invoices'.",
+                showIf: { mode: "from_field" },
               }),
               new Field({
                 name: "show_hidden",
@@ -104,17 +116,35 @@ async function get_state_fields(table_id, viewname, { mode, path_field }) {
 }
 
 /**
- * Given the view configuration + a Saltcorn row, compute the effective
- * relative path (below base_path) that the tree should start from.
+ * Compose the start path (relative to the plugin base_path) from three
+ * optional parts:
+ *   1. `view_base_path`  — static, per-view prefix
+ *   2. row field value    — only in "from_field" mode
+ *   3. `extra_subpath`   — static suffix, only in "from_field" mode
+ *
+ * Empty parts are dropped and slash-trimmed. The final joined path is
+ * passed through sanitizeRelativePath so traversal / absolute paths /
+ * backslashes are rejected consistently.
  */
 function computeStartPath(configuration, row) {
   const parts = [];
+  if (configuration.view_base_path) {
+    parts.push(String(configuration.view_base_path));
+  }
   if (configuration.mode === "from_field" && configuration.path_field && row) {
     const v = row[configuration.path_field];
     if (v) parts.push(String(v));
   }
-  if (configuration.extra_subpath) parts.push(String(configuration.extra_subpath));
-  const joined = parts.join("/");
+  if (
+    configuration.mode === "from_field" &&
+    configuration.extra_subpath
+  ) {
+    parts.push(String(configuration.extra_subpath));
+  }
+  const joined = parts
+    .map((p) => p.replace(/^\/+|\/+$/g, ""))
+    .filter(Boolean)
+    .join("/");
   // sanitizeRelativePath throws on traversal – we let the caller handle it.
   return sanitizeRelativePath(joined);
 }