fix: remove trailing slashes for fs.open to address Node v23 bug

rvagg · rvagg · commit 4ff6f98bc54e · 2024-10-25T16:18:19.000+11:00
Ref: nodejs/node#55527
diff --git a/st.js b/st.js
@@ -223,7 +223,10 @@ class Mount {
 
   // get a path from a url
   getPath (u) {
-    return path.join(this.path, u)
+    // trailing slash removal to fix Node.js v23 bug
+    // https://github.com/nodejs/node/pull/55527
+    // can be removed when this is resolved and released
+    return path.join(this.path, u.replace(/\/+$/, ''))
   }
 
   // get a url from a path
diff --git a/test/basic.js b/test/basic.js
@@ -97,6 +97,14 @@ test('multiball!', (t) => {
   }
 })
 
+test('trailing slash', (t) => {
+  req('/test/test/fixtures/', (er, res, body) => {
+    t.equal(res.statusCode, 200)
+    t.ok(/<html>.*Index of \/test\/fixtures<[\s\S]+index\.html[\s\S]+space in filename\.txt[\s\S]+<\/html>/.test(body.toString()))
+    t.end()
+  })
+})
+
 test('space in filename', (t) => {
   req('/test/test/fixtures/space in filename.txt', (er, res, body) => {
     t.equal(res.statusCode, 200)
