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

[oon00b]

fix: #3107

This PR may be related to issue #2200 .

The author should do the following, if applicable

• Add tests
• Run tests
• bun run format:fix && bun run lint:fix to format the code
• Add TSDoc/JSDoc to document the code

[codecov]

Codecov Report

Attention: Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.87%. Comparing base (97ec652) to head (7196c96).
Report is 1 commits behind head on main.

Files	Patch %	Lines
src/adapter/deno/serve-static.ts	0.00%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3108   +/-   ##
=======================================
  Coverage   95.86%   95.87%           
=======================================
  Files         137      137           
  Lines       13510    13511    +1     
  Branches     2383     2285   -98     
=======================================
+ Hits        12952    12953    +1     
  Misses        558      558           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[oon00b]

Sorry, the similar PR already existed in honojs/node-sever. I was missing it because I have never used Node.js.

Can I close my PR?

[yusukebe]

Hi @oon00b

We're not sure we will accept this PR, but regarding supporting the absolute path, this will be a new feature. There are some concerns, so please give us a time.

[schettn]

If anyone has this error, this is my workaround:

 const relativePathToScript = path.relative(process.cwd(), __dirname)

  app.get(
    '/__studio/*',
    serveStatic({
      root: `${relativePathToScript}/../studio/dist`,

      rewriteRequestPath: path => {
        return path.replace('/__studio', '')
      },
      onNotFound: (path, c) => {
        console.log('not found', path, c.req.url)
      }
    })