fix(filepath): allow suffix includes - and _ (honojs#2910)

nakasyou · Jun 6, 2024 · cb55866 · cb55866
1 parent 817626f
commit cb55866
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/utils/filepath.test.ts b/src/utils/filepath.test.ts
@@ -56,6 +56,9 @@ describe('getFilePath', () => {
     expect(getFilePath({ filename: 'foo', root: 'bar', defaultDocument: 'index.txt' })).toBe(
       'bar/foo/index.txt'
     )
+
+    expect(getFilePath({ filename: 'filename.suffix_index' })).toBe('filename.suffix_index')
+    expect(getFilePath({ filename: 'filename.suffix-index' })).toBe('filename.suffix-index')
   })
 })
 

diff --git a/src/utils/filepath.ts b/src/utils/filepath.ts
@@ -16,7 +16,7 @@ export const getFilePath = (options: FilePathOptions): string | undefined => {
   if (filename.endsWith('/')) {
     // /top/ => /top/index.html
     filename = filename.concat(defaultDocument)
-  } else if (!filename.match(/\.[a-zA-Z0-9]+$/)) {
+  } else if (!filename.match(/\.[a-zA-Z0-9_-]+$/)) {
     // /top => /top/index.html
     filename = filename.concat('/' + defaultDocument)
   }