saltcorn-samba 0.4.4 → 0.4.6

package/CHANGELOG.md

@@ -4,6 +4,62 @@ 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.6] – 2026-07-05
+
+### Fixed – **Schreibvarianten-Test verdoppelte den Basispfad**
+
+Die Case-Probes in v0.4.4/0.4.5 riefen `client.readdir(<candidate>)` auf.
+Der Wrapper prependiert aber automatisch den bereits konfigurierten
+`basePath` — dadurch wurde intern nach `buero/static/static` statt
+`buero/static` gesucht. Alle Probes scheiterten deshalb systematisch
+mit derselben Meldung, unabhängig von der wahren Ursache.
+
+**Fix:** Die Probes gehen jetzt direkt gegen den rohen `smb3-client`
+(`client._raw.readdir(share/<absoluter Pfad>)`) und bauen die
+From-Share-Root-Pfade selbst zusammen. Der Test-Endpunkt zeigt jetzt
+zusätzlich pro Probe den tatsächlich getesteten Pfad an.
+
+### Fixed – **Falscher „Login rejected“-Hinweis bei BASE_PATH_NOT_FOUND**
+
+Der Fallback-Hint-Table prüfte per Substring-Match auf `access_denied`
+u.\u00e4. — das triggerte auf den erklärenden Text in unserer eigenen
+deutschen Fehlermeldung (wir *erwähnen* `ACCESS_DENIED` didaktisch).
+Ein ungefundener Basispfad wurde deshalb zusätzlich mit dem irreführenden
+Hinweis „Login rejected“ versehen.
+
+**Fix:** Fehler mit eigenem `code` (`BASE_PATH_NOT_FOUND`,
+`BASE_PATH_NOT_A_DIR`) überspringen den Substring-Match — die
+Wrapper-Meldung ist bereits vollständig und braucht keinen „Hint“.
+
+### Changed – **Bessere Basispfad-Meldung**
+
+Wenn kein Case-Match gefunden wurde, zeigt die Meldung jetzt den
+tatsächlich getesteten UNC-Pfad (`\\\\server\\share\\pfad`) und weist
+explizit darauf hin, dass der Basispfad **relativ zur Freigabe** ist —
+also `unterordner` und **nicht** `sharename/unterordner`.
+
+## [0.4.5] – 2026-07-05
+
+### Fixed – **Test-Verbindung-Button reagiert nicht mehr (v0.4.4-Regression)**
+
+In v0.4.4 enthielt der neu hinzugefügte Diagnose-IIFE-Block im Inline-
+Browser-Script eine Regex `.split(/[\r\n]/)`. Der HTML-Block ist ein
+**Template-Literal** in `index.js`; darin werden `\r` und `\n` zu
+**echten Steuerzeichen** aufgelöst. Das gerenderte `<script>` enthielt
+deshalb einen literalen Zeilenumbruch mitten in der Regex — der Browser
+konnte das Skript nicht mehr parsen, und `window.sambaTestConn` wurde
+nie definiert. Ein Klick auf „Verbindung jetzt testen“ zeigte deshalb
+keine Reaktion.
+
+**Fix:** Die Escape-Sequenz im Source auf `\\r\\n` verdoppelt, damit im
+gerenderten Skript wieder `\r\n` steht (also die Zeichenklasse, nicht
+die literalen Zeichen). Der Skript-Parse funktioniert wieder, die
+Test-Schaltfläche reagiert.
+
+Zusätzlich wurde die Rendering-Pipeline mit einem Node-`--check` auf
+dem *extrahierten* Browser-Skript verifiziert, damit dieser konkrete
+Fehlermodus in Zukunft schon lokal auffällt.
+
 ## [0.4.4] – 2026-07-05
 
 ### Fixed – **Falscher `NAME_NOT_FOUND` durch `stat()`-Vorprüfung**

package/index.js

@@ -234,7 +234,7 @@ window.sambaTestConn = async function(btn) {
               box += '<details style="margin-top:.3rem"><summary>Schreibvarianten-Test</summary><table class="table table-sm" style="margin-top:.3rem;font-size:.85em">';
               box += '<thead><tr><th>Variante</th><th>Ergebnis</th></tr></thead><tbody>';
               d.spelling_probes.forEach(function(p){
-                box += '<tr><td><code>' + esc(p.candidate) + '</code></td><td>' + (p.ok ? '✓ auflistbar' : '✗ <span class="text-muted">' + esc(String(p.error||'').split(/[\r\n]/)[0].slice(0,140)) + '</span>') + '</td></tr>';
+                box += '<tr><td><code>' + esc(p.candidate) + '</code></td><td>' + (p.ok ? '✓ auflistbar' : '✗ <span class="text-muted">' + esc(String(p.error||'').split(/[\\r\\n]/)[0].slice(0,140)) + '</span>') + '</td></tr>';
               });
               box += '</tbody></table></details>';
             }
@@ -901,15 +901,28 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
               const missing = parts[parts.length - 1];
               const parent = parts.slice(0, -1).join("/");
               const parentAbs = parent || "(Share-Root)";
+              // We must probe via the *raw* smb3-client because the
+              // wrapper's readdir() automatically prepends basePath —
+              // that would double up the segment we're trying to test
+              // (e.g. readdir("static") → "buero/static/static").
+              const raw = client._raw;
+              const rawShare = client.shareName;
+              async function rawReaddir(relFromShareRoot) {
+                const full = [rawShare, relFromShareRoot].filter(Boolean).join("/");
+                return await raw.readdir(full, { withFileTypes: true });
+              }
               let siblings = null;
               let parent_error = null;
               if (parent) {
                 try {
-                  const listing = await client.readdir(parent);
+                  // parent here is relative to basePath — but we want
+                  // absolute-from-share-root paths in the raw call.
+                  const parentFromShare = [client.basePath, parent].filter(Boolean).join("/");
+                  const listing = await rawReaddir(parentFromShare);
                   siblings = Array.isArray(listing)
                     ? listing.map((d) => ({
-                        name: d && (d.name || d),
-                        isDirectory: !!(d && (d.isDirectory === true || (typeof d.isDirectory === "function" && d.isDirectory()))),
+                        name: d && d.name,
+                        isDirectory: !!(d && typeof d.isDirectory === "function" && d.isDirectory()),
                       }))
                     : null;
                 } catch (parentErr) {
@@ -932,42 +945,44 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
               addProbe(missing.charAt(0).toUpperCase() + missing.slice(1).toLowerCase());
               const probe_results = [];
               for (const p of probes) {
-                const candidate = parent ? parent + "/" + p : p;
+                // Build the *from-share-root* relative path so the probe
+                // does not accidentally get basePath prepended twice.
+                const relFromShare = [client.basePath, parent, p].filter(Boolean).join("/");
                 let ok = false;
                 let perr = null;
                 try {
-                  await client.readdir(candidate);
+                  await rawReaddir(relFromShare);
                   ok = true;
                 } catch (pErr) {
                   perr = String((pErr && pErr.message) || pErr || "");
                 }
-                probe_results.push({ candidate: p, ok, error: ok ? null : perr });
+                probe_results.push({ candidate: p, tested_path: relFromShare, ok, error: ok ? null : perr });
               }
               const workingAlt = probe_results.find(
                 (r) => r.ok && r.candidate !== missing
               );
+              const testedPath = "\\\\" + testCfg.server + "\\" + testCfg.share + "\\" + rel.replace(/\//g, "\\");
               let hintText;
               if (workingAlt) {
                 hintText =
-                  "Der Ordner heisst auf dem Server \u201e" +
+                  "Der Ordner hei\u00dft auf dem Server \u201e" +
                   workingAlt.candidate +
-                  "\u201c (andere Gross-/Kleinschreibung). Bitte den " +
+                  "\u201c (andere Gro\u00df-/Kleinschreibung). Bitte den " +
                   "Basispfad exakt so eintragen \u2014 der Samba-Server ist " +
                   "case-sensitive (\u201ecase sensitive = yes\u201c in smb.conf).";
               } else {
                 hintText =
-                  "Der Basispfad \u201e" + rel + "\u201c ist auf dem " +
-                  "Server nicht auffindbar. M\u00f6gliche Ursachen: " +
-                  "(a) der Ordner existiert wirklich nicht (bitte mit " +
-                  "einem SMB-Client wie \u201esmbclient\u201c oder dem " +
-                  "Windows-Explorer gegenpr\u00fcfen); " +
+                  "Der Basispfad \u201e" + rel + "\u201c wurde auf der " +
+                  "Freigabe \u201e" + testCfg.share + "\u201c nicht " +
+                  "gefunden. Getestet wurde der Pfad: " + testedPath + ". " +
+                  "M\u00f6gliche Ursachen: (a) Ordner existiert unter genau " +
+                  "diesem Namen nicht in dieser Freigabe (Basispfad ist " +
+                  "relativ zur Freigabe \u2014 also z.\u202fB. \u201eunterordner\u201c, " +
+                  "nicht \u201e" + testCfg.share + "/unterordner\u201c); " +
                   "(b) der angemeldete Benutzer \u201e" +
                   (testCfg.username || "(anonymous)") +
-                  "\u201c hat kein Leserecht auf den Ordner (Samba antwortet " +
-                  "dann bei \u201ehide unreadable = yes\u201c mit " +
-                  "NAME_NOT_FOUND statt ACCESS_DENIED); " +
-                  "(c) der Ordner ist per \u201eveto files\u201c / " +
-                  "\u201ehide files\u201c auf dem Server ausgeblendet.";
+                  "\u201c darf den Ordner nicht sehen; " +
+                  "(c) der Ordner ist per Samba-Konfiguration ausgeblendet.";
               }
               const e = new Error(hintText);
               e.cause = err;
@@ -1017,7 +1032,19 @@ code{background:#f4f4f4;padding:2px 6px;border-radius:3px;word-break:break-all}<
         const code = (err && (err.code || err.errno)) || null;
         let hint = null;
         const m = msg.toLowerCase();
-        if (code === "ECONNREFUSED" || m.includes("econnrefused"))
+        // If the wrapper already produced a well-formed, self-explanatory
+        // error (own `code` set), skip the substring-match hint table —
+        // it would otherwise trigger on words that appear inside our own
+        // human-readable German explanation text (e.g. our message text
+        // mentions "access_denied" or "path" for teaching purposes).
+        const codesWithOwnMessage = new Set([
+          "BASE_PATH_NOT_FOUND",
+          "BASE_PATH_NOT_A_DIR",
+        ]);
+        if (code && codesWithOwnMessage.has(code)) {
+          hint = null;
+        }
+        else if (code === "ECONNREFUSED" || m.includes("econnrefused"))
           hint = "The server refused the connection on port " + testCfg.port +
             ". Check that Samba is running and that a firewall (or Docker) does not block TCP/445.";
         else if (code === "ETIMEDOUT" || m.includes("etimedout") || m.includes("timed out"))

package/package.json

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