fix(serveStatic): support serving file from absolute path for Deno and Bun

runtime_tests/bun/abs-static/plain.txt

@@ -0,0 +1 @@
+Bun!!!

runtime_tests/bun/index.test.tsx

@@ -106,6 +106,12 @@ describe('Serve Static Middleware', () => {
       rewriteRequestPath: (path) => path.replace(/^\/dot-static/, './.static'),
     })
   )
+  app.get(
+    '/abs-static/*',
+    serveStatic({
+      root: `${process.cwd()}/runtime_tests/bun/`,
+    })
+  )
 
   beforeEach(() => onNotFound.mockClear())
 
@@ -145,6 +151,12 @@ describe('Serve Static Middleware', () => {
     expect(res.status).toBe(200)
     expect(await res.text()).toBe('Bun!!')
   })
+
+  it('Should return 200 response - /abs-static/plain.txt', async () => {
+    const res = await app.request(new Request('http://localhost/abs-static/plain.txt'))
+    expect(res.status).toBe(200)
+    expect(await res.text()).toBe('Bun!!!')
+  })
 })
 
 // Bun support WebCrypto since v0.2.2

runtime_tests/deno/abs-static/plain.txt

@@ -0,0 +1 @@
+Deno!!!

runtime_tests/deno/middleware.test.tsx

@@ -93,6 +93,13 @@ Deno.test('Serve Static middleware', async () => {
     })
   )
 
+  app.get(
+    '/abs-static/*',
+    serveStatic({
+      root: await Deno.realPath('./runtime_tests/deno'),
+    })
+  )
+
   let res = await app.request('http://localhost/favicon.ico')
   assertEquals(res.status, 200)
   assertEquals(res.headers.get('Content-Type'), 'image/x-icon')
@@ -117,6 +124,10 @@ Deno.test('Serve Static middleware', async () => {
   assertEquals(await res.text(), 'Deno!!')
   assertSpyCalls(onNotFound, 1)
 
+  res = await app.request('http://localhost/abs-static/plain.txt')
+  assertEquals(res.status, 200)
+  assertEquals(await res.text(), 'Deno!!!')
+
   res = await app.fetch({
     method: 'GET',
     url: 'http://localhost/static/%2e%2e/static/plain.txt',

src/adapter/bun/serve-static.ts

@@ -8,13 +8,12 @@ export const serveStatic = <E extends Env = Env>(
 ): MiddlewareHandler => {
   return async function serveStatic(c, next) {
     const getContent = async (path: string) => {
-      path = `./${path}`
       // @ts-ignore
       const file = Bun.file(path)
       return (await file.exists()) ? file : null
     }
-    const pathResolve = (path: string) => {
-      return `./${path}`
+    const pathResolve = (path: string, isAbsolutePath?: boolean) => {
+      return isAbsolutePath ? `/${path}` : `./${path}`
     }
     return baseServeStatic({
       ...options,

src/adapter/deno/serve-static.ts

@@ -19,8 +19,8 @@ export const serveStatic = <E extends Env = Env>(
         console.warn(`${e}`)
       }
     }
-    const pathResolve = (path: string) => {
-      return `./${path}`
+    const pathResolve = (path: string, isAbsolutePath?: boolean) => {
+      return isAbsolutePath ? `/${path}` : `./${path}`
     }
     return baseServeStatic({
       ...options,

src/middleware/serve-static/index.ts

@@ -25,7 +25,7 @@ const defaultPathResolve = (path: string) => path
 export const serveStatic = <E extends Env = Env>(
   options: ServeStaticOptions<E> & {
     getContent: (path: string, c: Context<E>) => Promise<Data | Response | null>
-    pathResolve?: (path: string) => string
+    pathResolve?: (path: string, isAbsolutePath?: boolean) => string
   }
 ): MiddlewareHandler => {
   return async (c, next) => {
@@ -50,7 +50,9 @@ export const serveStatic = <E extends Env = Env>(
     }
 
     const getContent = options.getContent
-    const pathResolve = options.pathResolve ?? defaultPathResolve
+    const pathResolve = (p: string) => {
+      return (options.pathResolve ?? defaultPathResolve)(p, root?.startsWith('/'))
+    }
 
     path = pathResolve(path)
     let content = await getContent(path, c)