http-proxy-middleware 3.0.4 → 3.0.5

package/dist/handlers/fix-request-body.d.ts

@@ -1,9 +1,8 @@
 import type * as http from 'http';
-import type { Options } from '../types';
 export type BodyParserLikeRequest = http.IncomingMessage & {
     body?: any;
 };
 /**
  * Fix proxied body if bodyParser is involved.
  */
-export declare function fixRequestBody<TReq extends BodyParserLikeRequest = BodyParserLikeRequest>(proxyReq: http.ClientRequest, req: TReq, res: http.ServerResponse<http.IncomingMessage>, options: Options): void;
+export declare function fixRequestBody<TReq extends BodyParserLikeRequest = BodyParserLikeRequest>(proxyReq: http.ClientRequest, req: TReq): void;

package/dist/handlers/fix-request-body.js

@@ -2,11 +2,14 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.fixRequestBody = fixRequestBody;
 const querystring = require("querystring");
-const logger_1 = require("../logger");
 /**
  * Fix proxied body if bodyParser is involved.
  */
-function fixRequestBody(proxyReq, req, res, options) {
+function fixRequestBody(proxyReq, req) {
+    // skip fixRequestBody() when req.readableLength not 0 (bodyParser failure)
+    if (req.readableLength !== 0) {
+        return;
+    }
     const requestBody = req.body;
     if (!requestBody) {
         return;
@@ -15,19 +18,6 @@ function fixRequestBody(proxyReq, req, res, options) {
     if (!contentType) {
         return;
     }
-    const logger = (0, logger_1.getLogger)(options);
-    // Handle bad request when unexpected "Connect: Upgrade" header is provided
-    if (/upgrade/gi.test(proxyReq.getHeader('Connection'))) {
-        handleBadRequest({ proxyReq, req, res });
-        logger.error(`[HPM] HPM_UNEXPECTED_CONNECTION_UPGRADE_HEADER. Aborted request: ${req.url}`);
-        return;
-    }
-    // Handle bad request when invalid request body is provided
-    if (hasInvalidKeys(requestBody)) {
-        handleBadRequest({ proxyReq, req, res });
-        logger.error(`[HPM] HPM_INVALID_REQUEST_DATA. Aborted request: ${req.url}`);
-        return;
-    }
     const writeBody = (bodyData) => {
         proxyReq.setHeader('Content-Length', Buffer.byteLength(bodyData));
         proxyReq.write(bodyData);
@@ -58,12 +48,3 @@ function handlerFormDataBodyData(contentType, data) {
     }
     return str;
 }
-function hasInvalidKeys(obj) {
-    return Object.keys(obj).some((key) => /[\n\r]/.test(key));
-}
-function handleBadRequest({ proxyReq, req, res }) {
-    res.writeHead(400);
-    res.end('Bad Request');
-    proxyReq.destroy();
-    req.destroy();
-}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "http-proxy-middleware",
   "type": "commonjs",
-  "version": "3.0.4",
+  "version": "3.0.5",
   "description": "The one-liner node.js proxy middleware for connect, express, next.js and more",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",