hapi-terminator 0.1.0 → 0.2.0

package/README.md

@@ -1,6 +1,6 @@
 # hapi-terminator
 
-A Hapi plugin that terminates requests with payloads that exceed a specified size limit. This plugin helps protect your server from excessively large payloads by destroying the socket connection before the entire payload is processed.
+A Hapi plugin that terminates requests with payloads that exceed a specified size limit. This plugin helps protect your server from excessively large payloads by gracefully ending the socket connection before the entire payload is processed.
 
 ## Features
 
@@ -127,8 +127,8 @@ You can configure per-route limits using the route options:
 
 ### Behavior
 
-- **Registered Routes**: When a payload exceeds the limit on a registered route, the socket is destroyed and a `413 Payload Too Large` error is thrown.
-- **Unregistered Routes**: When a payload exceeds the limit on an unregistered route, the socket is destroyed and a `404 Not Found` error is thrown.
+- **Registered Routes**: When a payload exceeds the limit on a registered route, the socket is gracefully ended and a `413 Payload Too Large` error is returned.
+- **Unregistered Routes**: When a payload exceeds the limit on an unregistered route, the socket is gracefully ended and a `404 Not Found` error is returned.
 - **Per-Route Limits**: Route-specific limits take precedence over global limits, allowing you to customize limits for individual routes.
 - **Disabled**: Set to `null` or `undefined` to disable termination for that category or route.
 
@@ -136,8 +136,8 @@ You can configure per-route limits using the route options:
 
 The plugin hooks into Hapi's `onRequest` extension point and checks the `Content-Length` header of incoming requests. If the content length exceeds the configured threshold:
 
-1. The socket connection is immediately destroyed
-2. An appropriate error response is thrown (413 for registered routes, 404 for unregistered routes)
+1. An appropriate error response is returned (413 for registered routes, 404 for unregistered routes)
+2. The socket connection is gracefully ended after the response is sent
 3. No further processing occurs, saving server resources
 
 ## License

package/dist/index.d.ts

@@ -28,7 +28,6 @@ export declare const plugin: {
             "typescript-eslint": string;
         };
         dependencies: {
-            "@hapi/boom": string;
             zod: string;
         };
         prettier: string;
@@ -91,7 +90,6 @@ declare const _default: {
                 "typescript-eslint": string;
             };
             dependencies: {
-                "@hapi/boom": string;
                 zod: string;
             };
             prettier: string;

package/dist/index.js

@@ -1,5 +1,4 @@
 import assert from 'node:assert';
-import Boom from '@hapi/boom';
 import { z } from 'zod/mini';
 import pkg from './package.json';
 const LIMIT_OPTION_NAMES = {
@@ -31,17 +30,23 @@ function validateHookHandler(pluginOptions, routeOptionsCache) {
         const { route, options } = getRouteAndOptions(request, routeOptionsCache) ?? {};
         if (route != null) {
             return handler(contentLength, LIMIT_OPTION_NAMES.REGISTERED, options, option => {
-                throw Boom.entityTooLarge(`Payload content length greater than maximum allowed: ${option}`);
+                return h
+                    .response({
+                    error: 'Request Entity Too Large',
+                    message: `Payload content length greater than maximum allowed: ${option}`,
+                    statusCode: 413,
+                })
+                    .code(413);
             });
         }
         assert(options == null, "Unregistered routes can't have route options");
         return handler(contentLength, LIMIT_OPTION_NAMES.UNREGISTERED, null, () => {
-            throw Boom.notFound();
+            return h.response({ error: 'Not Found', message: 'Not Found', statusCode: 404 }).code(404);
         });
     };
 }
 function getRouteAndOptions(request, routeOptionsCache) {
-    const matchedRoute = request.server.match(request.method, request.path);
+    const matchedRoute = request.server.match(request.method, request.path, request.info.host);
     if (matchedRoute == null) {
         return null;
     }
@@ -55,17 +60,20 @@ function getRouteAndOptions(request, routeOptionsCache) {
     return { options, route: matchedRoute };
 }
 function validateRoute(request, h, options) {
-    return (contentLength, optionName, routeOptions, throwError) => {
+    return (contentLength, optionName, routeOptions, response) => {
         const option = options?.[optionName];
         const limit = routeOptions?.[PACKAGE_NAME]?.limit ?? option;
         if (limit == null) {
             return h.continue;
         }
-        if (contentLength > limit) {
-            request.raw.req.socket?.destroy();
-            throwError(limit);
+        if (contentLength <= limit) {
+            return h.continue;
         }
-        return h.continue;
+        const result = response(limit).takeover();
+        request.raw.res.once('finish', () => {
+            request.raw.req.socket.end();
+        });
+        return result;
     };
 }
 function getRoutePluginSettings(matchedRoute) {

package/dist/package.json

@@ -3,7 +3,7 @@
     "main": "dist/index.js",
     "typings": "dist/index.d.ts",
     "type": "module",
-    "version": "0.1.0",
+    "version": "0.2.0",
     "license": "MIT",
     "author": {
         "name": "Kamaal Farah"
@@ -23,7 +23,6 @@
         "typescript-eslint": "^8.52.0"
     },
     "dependencies": {
-        "@hapi/boom": "^10.0.1",
         "zod": "^4.3.5"
     },
     "prettier": "@kamaalio/prettier-config",

package/package.json

@@ -3,7 +3,7 @@
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",
   "type": "module",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "license": "MIT",
   "author": {
     "name": "Kamaal Farah"
@@ -23,7 +23,6 @@
     "typescript-eslint": "^8.52.0"
   },
   "dependencies": {
-    "@hapi/boom": "^10.0.1",
     "zod": "^4.3.5"
   },
   "prettier": "@kamaalio/prettier-config",