saltcorn-samba 0.4.11 → 0.4.13

package/CHANGELOG.md

@@ -4,6 +4,107 @@ 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.13] – 2026-07-06
+
+### Fixed – **QUERY_DIRECTORY leere Patterns → 0xC0000033 (die eigentliche Ursache)**
+
+Mit v0.4.12 wurde ein Wire-Bug in smb3-client (FileNameOffset) gefixt,
+aber Samba lehnte weiterhin ab. Mit dem neuen `tools/diag-wire.js` konnte
+die tatsächliche Ursache byteweise verifiziert werden:
+
+**Bug:** smb3-client's `readdirAll` sendet ab der 2. Enumeration-Seite
+`searchPattern=""` (leerer String). Windows toleriert das, aber Samba's
+`source3/smbd/smb2_query_directory.c` enthält den strikten Check:
+
+```c
+if (state->in_file_name[0] == '\0') {
+    tevent_req_nterror(req, NT_STATUS_OBJECT_NAME_INVALID);
+    return tevent_req_post(req, ev);
+}
+```
+
+Das ergibt exakt den STATUS_OBJECT_NAME_INVALID (0xC0000033), den wir
+seit v0.4.0 sehen.
+
+**Fix in `readdir-compat.js`:** Der eigene Enumeration-Loop sendet auf
+**jeder** Seite `searchPattern="*"`, nicht nur beim ersten Request.
+`RESTART_SCANS` wird nur beim ersten Request gesetzt; ab dann sendet
+Samba nach dem letzten Batch korrekt STATUS_NO_MORE_FILES, was den Loop
+sauber terminiert. Verifiziert via `diag-wire.js`:
+
+- Probe 1 (pat=`*`, RESTART): STATUS_SUCCESS, 8 Einträge
+- Probe 2 (pat=`*`, ohne RESTART): STATUS_NO_MORE_FILES → Loop-Ende
+- Probe 6 (pat=`""`, offset=0): **0xC0000033** ← der Bug
+
+### Added
+
+- `tools/diag-wire.js` – Wire-Level-Diagnose mit Hex-Dump der SMB2-Bytes,
+  testet 6 verschiedene QUERY_DIRECTORY-Varianten (Info-Class, Buffer-
+  Größe, mit/ohne Pattern). Nützlich für zukünftige Kompatibilitäts-
+  probleme mit anderen SMB-Servern.
+
+### Upstream-Report aktualisiert
+
+`smb3-client-bug-report.md` beschreibt jetzt beide Bugs (FileNameOffset
+**und** leeres Pattern gegen Samba). Beide Fixes sind identisch simpel:
+auf jeder Page `*` senden.
+
+## [0.4.12] – 2026-07-06
+
+### Fixed – **`0xC0000033` beim readdir gegen Samba 4.23 endgültig behoben (Root Cause identifiziert)**
+
+Nach ausführlicher Wire-Level-Analyse (siehe `smb-diag-report.txt`
+aus 0.4.11) ist die eigentliche Ursache identifiziert:
+
+**Bug in `smb3-client@0.2.0`, Datei `dist/wire/structs/queryDirectory.js`:**
+Der Encoder für SMB2 QUERY_DIRECTORY setzt das `FileNameOffset`-Feld
+immer hart auf 96, auch wenn `FileNameLength = 0` gesendet wird
+(z. B. auf der 2. und folgenden Enumeration-Seite).
+[MS-SMB2 §2.2.33](https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-smb2/10906442-294c-46d3-8515-c277efe1f752)
+verlangt für diesen Fall **`FileNameOffset = 0`**.
+Windows Server toleriert die Fehlbelegung, Samba 4.23 lehnt sie
+strikt mit `STATUS_OBJECT_NAME_INVALID` (`0xC0000033`) ab —
+deshalb funktioniert nichts, was das Listing über mehrere Pages
+braucht, wovon `smb3-client` grundsätzlich ausgeht.
+
+### Fix
+
+Die frühere `readdir-compat.js` (die 3 FileInformationClass-Werte
+durchprobierte) ist ersetzt durch einen echten **Wire-Format-Patch**:
+
+- Eigener spec-konformer `encodeQueryDirectoryRequest`-Encoder in
+  `readdir-compat.js`
+- Eigene QUERY_DIRECTORY-Loop, die den gepatchten Encoder verwendet
+- Wiederverwendung von `Open.withOpen` aus `smb3-client` (Open, Close,
+  Tree-Connect bleiben unverändert) via dynamischem ESM-Import über
+  `file://` URLs (das `exports`-Gate von `smb3-client` sperrt sonst
+  jeden Subpath-Import)
+- `smb-client.js` nutzt jetzt ausschließlich `readdirCompat` als
+  readdir-Pfad. Kein Fallback mehr auf das kaputte `client.readdir()`.
+- Bonus: Die Rich-Dirents aus dem gepatchten QUERY_DIRECTORY liefern
+  Name, Größe, mtime und ctime in einem Roundtrip. Der bisherige
+  Fan-out mit einem `stat()`-Aufruf pro Eintrag entfällt — große
+  Verzeichnisse werden dadurch **deutlich schneller**.
+
+### Bekannt – Upstream-Fix eingereicht
+
+Parallel wurde ein Bug-Report an `smb3-client` (GitHub:
+`euricojardim/smb3-client`) vorbereitet inklusive Reproduktions-
+Diagnose und Patch-Vorschlag. Sobald der Upstream-Fix in einer
+neuen `smb3-client`-Version verfügbar ist, kann dieses Plugin die
+`readdir-compat.js` wieder entfernen und die eingebaute API direkt
+nutzen.
+
+### Migration – keine Konfigurationsänderung nötig
+
+Der Fix greift automatisch. Der gelbe Hinweis „Basispfad bestätigt,
+aber Auflisten funktioniert nicht“ aus 0.4.9–0.4.11 fällt weg, weil
+das Auflisten jetzt tatsächlich funktioniert. Der File-Manager
+zeigt Ordner-Inhalte, die Baum-Ansicht rendert Verzeichnis-Bäume,
+PDF-Ansicht und Datei-Downloads funktionieren unverändert.
+
+---
+
 ## [0.4.11] – 2026-07-06
 
 ### Changed – **Ehrliche Fehlerpropagation statt stiller Fallback**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "saltcorn-samba",
-  "version": "0.4.11",
+  "version": "0.4.13",
   "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": {
@@ -44,6 +44,7 @@
     "filemanager-view.js",
     "pdf-view.js",
     "public/",
+    "tools/",
     "README.md",
     "CHANGELOG.md",
     "LICENSE"