fix: support more than one path params (#61)

honojs · Feb 16, 2024 · 60a0d0b · 60a0d0b
1 parent a3e872f
commit 60a0d0b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/utils/file.ts b/src/utils/file.ts
@@ -5,7 +5,7 @@ export const filePathToPath = (filePath: string) => {
     .replace(/^\/?index$/, '/') // `/index`
     .replace(/\/index$/, '') // `/about/index`
     .replace(/\[\.{3}.+\]/, '*')
-    .replace(/\[(.+)\]/, ':$1')
+    .replace(/\[(.+?)\]/g, ':$1')
   return /^\//.test(filePath) ? filePath : '/' + filePath
 }
 

diff --git a/test/unit/utils/file.test.ts b/test/unit/utils/file.test.ts
@@ -27,6 +27,7 @@ describe('filePathToPath', () => {
     expect(filePathToPath('/about/[name].tsx')).toBe('/about/:name')
     expect(filePathToPath('/about/[...foo].tsx')).toBe('/about/*')
     expect(filePathToPath('/about/[name]/address.tsx')).toBe('/about/:name/address')
+    expect(filePathToPath('/about/[arg1]/[arg2]')).toBe('/about/:arg1/:arg2')
   })
 })