@hono/node-server 1.3.4 → 1.4.0

package/README.md

@@ -162,6 +162,22 @@ app.use(
 )
 ```
 
+#### `onNotFound`
+
+The `onNotFound` is useful for debugging. You can write a handle for when a file is not found.
+
+```ts
+app.use(
+  '/static/*',
+  serveStatic({
+    root: './non-existent-dir',
+    onNotFound: (path, c) => {
+      console.log(`${path} is not found, request to ${c.req.path}`)
+    },
+  })
+)
+```
+
 ## Related projects
 
 - Hono - <https://hono.dev>

package/dist/index.js

@@ -281,19 +281,12 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
-  if (!(res instanceof Response)) {
-    return handleResponseError(
-      // @ts-expect-error the object must have `toString()`
-      new Error(`The response is not an instance of Response, but ${res.toString()}`),
-      outgoing
-    );
-  }
-  if (cacheKey in res) {
-    try {
+  try {
+    if (cacheKey in res) {
       return responseViaCache(res, outgoing);
-    } catch (e) {
-      return handleResponseError(e, outgoing);
     }
+  } catch (e) {
+    return handleResponseError(e, outgoing);
   }
   const resHeaderRecord = buildOutgoingHttpHeaders(res.headers);
   if (res.body) {

package/dist/index.mjs

@@ -243,19 +243,12 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
-  if (!(res instanceof Response)) {
-    return handleResponseError(
-      // @ts-expect-error the object must have `toString()`
-      new Error(`The response is not an instance of Response, but ${res.toString()}`),
-      outgoing
-    );
-  }
-  if (cacheKey in res) {
-    try {
+  try {
+    if (cacheKey in res) {
       return responseViaCache(res, outgoing);
-    } catch (e) {
-      return handleResponseError(e, outgoing);
     }
+  } catch (e) {
+    return handleResponseError(e, outgoing);
   }
   const resHeaderRecord = buildOutgoingHttpHeaders(res.headers);
   if (res.body) {

package/dist/listener.js

@@ -276,19 +276,12 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
-  if (!(res instanceof Response)) {
-    return handleResponseError(
-      // @ts-expect-error the object must have `toString()`
-      new Error(`The response is not an instance of Response, but ${res.toString()}`),
-      outgoing
-    );
-  }
-  if (cacheKey in res) {
-    try {
+  try {
+    if (cacheKey in res) {
       return responseViaCache(res, outgoing);
-    } catch (e) {
-      return handleResponseError(e, outgoing);
     }
+  } catch (e) {
+    return handleResponseError(e, outgoing);
   }
   const resHeaderRecord = buildOutgoingHttpHeaders(res.headers);
   if (res.body) {

package/dist/listener.mjs

@@ -240,19 +240,12 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
-  if (!(res instanceof Response)) {
-    return handleResponseError(
-      // @ts-expect-error the object must have `toString()`
-      new Error(`The response is not an instance of Response, but ${res.toString()}`),
-      outgoing
-    );
-  }
-  if (cacheKey in res) {
-    try {
+  try {
+    if (cacheKey in res) {
       return responseViaCache(res, outgoing);
-    } catch (e) {
-      return handleResponseError(e, outgoing);
     }
+  } catch (e) {
+    return handleResponseError(e, outgoing);
   }
   const resHeaderRecord = buildOutgoingHttpHeaders(res.headers);
   if (res.body) {

package/dist/serve-static.d.mts

@@ -1,4 +1,4 @@
-import { MiddlewareHandler } from 'hono';
+import { Context, MiddlewareHandler } from 'hono';
 
 type ServeStaticOptions = {
     /**
@@ -8,6 +8,7 @@ type ServeStaticOptions = {
     path?: string;
     index?: string;
     rewriteRequestPath?: (path: string) => string;
+    onNotFound?: (path: string, c: Context) => void | Promise<void>;
 };
 declare const serveStatic: (options?: ServeStaticOptions) => MiddlewareHandler;
 

package/dist/serve-static.d.ts

@@ -1,4 +1,4 @@
-import { MiddlewareHandler } from 'hono';
+import { Context, MiddlewareHandler } from 'hono';
 
 type ServeStaticOptions = {
     /**
@@ -8,6 +8,7 @@ type ServeStaticOptions = {
     path?: string;
     index?: string;
     rewriteRequestPath?: (path: string) => string;
+    onNotFound?: (path: string, c: Context) => void | Promise<void>;
 };
 declare const serveStatic: (options?: ServeStaticOptions) => MiddlewareHandler;
 

package/dist/serve-static.js

@@ -171,6 +171,7 @@ var serveStatic = (options = { root: "" }) => {
       return next();
     path = `./${path}`;
     if (!(0, import_fs.existsSync)(path)) {
+      await options.onNotFound?.(path, c);
       return next();
     }
     const mimeType = getMimeType(path);

package/dist/serve-static.mjs

@@ -147,6 +147,7 @@ var serveStatic = (options = { root: "" }) => {
       return next();
     path = `./${path}`;
     if (!existsSync(path)) {
+      await options.onNotFound?.(path, c);
       return next();
     }
     const mimeType = getMimeType(path);

package/dist/server.js

@@ -278,19 +278,12 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
-  if (!(res instanceof Response)) {
-    return handleResponseError(
-      // @ts-expect-error the object must have `toString()`
-      new Error(`The response is not an instance of Response, but ${res.toString()}`),
-      outgoing
-    );
-  }
-  if (cacheKey in res) {
-    try {
+  try {
+    if (cacheKey in res) {
       return responseViaCache(res, outgoing);
-    } catch (e) {
-      return handleResponseError(e, outgoing);
     }
+  } catch (e) {
+    return handleResponseError(e, outgoing);
   }
   const resHeaderRecord = buildOutgoingHttpHeaders(res.headers);
   if (res.body) {

package/dist/server.mjs

@@ -243,19 +243,12 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
-  if (!(res instanceof Response)) {
-    return handleResponseError(
-      // @ts-expect-error the object must have `toString()`
-      new Error(`The response is not an instance of Response, but ${res.toString()}`),
-      outgoing
-    );
-  }
-  if (cacheKey in res) {
-    try {
+  try {
+    if (cacheKey in res) {
       return responseViaCache(res, outgoing);
-    } catch (e) {
-      return handleResponseError(e, outgoing);
     }
+  } catch (e) {
+    return handleResponseError(e, outgoing);
   }
   const resHeaderRecord = buildOutgoingHttpHeaders(res.headers);
   if (res.body) {

package/dist/vercel.js

@@ -276,19 +276,12 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
-  if (!(res instanceof Response)) {
-    return handleResponseError(
-      // @ts-expect-error the object must have `toString()`
-      new Error(`The response is not an instance of Response, but ${res.toString()}`),
-      outgoing
-    );
-  }
-  if (cacheKey in res) {
-    try {
+  try {
+    if (cacheKey in res) {
       return responseViaCache(res, outgoing);
-    } catch (e) {
-      return handleResponseError(e, outgoing);
     }
+  } catch (e) {
+    return handleResponseError(e, outgoing);
   }
   const resHeaderRecord = buildOutgoingHttpHeaders(res.headers);
   if (res.body) {

package/dist/vercel.mjs

@@ -240,19 +240,12 @@ var responseViaResponseObject = async (res, outgoing) => {
   if (res instanceof Promise) {
     res = await res.catch(handleFetchError);
   }
-  if (!(res instanceof Response)) {
-    return handleResponseError(
-      // @ts-expect-error the object must have `toString()`
-      new Error(`The response is not an instance of Response, but ${res.toString()}`),
-      outgoing
-    );
-  }
-  if (cacheKey in res) {
-    try {
+  try {
+    if (cacheKey in res) {
       return responseViaCache(res, outgoing);
-    } catch (e) {
-      return handleResponseError(e, outgoing);
     }
+  } catch (e) {
+    return handleResponseError(e, outgoing);
   }
   const resHeaderRecord = buildOutgoingHttpHeaders(res.headers);
   if (res.body) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hono/node-server",
-  "version": "1.3.4",
+  "version": "1.4.0",
   "description": "Node.js Adapter for Hono",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -66,6 +66,7 @@
     "@types/jest": "^29.5.3",
     "@types/node": "^20.10.0",
     "@types/supertest": "^2.0.12",
+    "@whatwg-node/fetch": "^0.9.14",
     "eslint": "^8.55.0",
     "hono": "^3.11.7",
     "jest": "^29.6.1",