http-proxy-middleware 2.0.8 → 2.0.9

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

@@ -3,4 +3,4 @@ import type * as http from 'http';
 /**
  * Fix proxied body if bodyParser is involved.
  */
-export declare function fixRequestBody(proxyReq: http.ClientRequest, req: http.IncomingMessage, res: http.ServerResponse): void;
+export declare function fixRequestBody(proxyReq: http.ClientRequest, req: http.IncomingMessage): void;

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

@@ -5,7 +5,11 @@ const querystring = require("querystring");
 /**
  * Fix proxied body if bodyParser is involved.
  */
-function fixRequestBody(proxyReq, req, res) {
+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;
@@ -14,16 +18,6 @@ function fixRequestBody(proxyReq, req, res) {
     if (!contentType) {
         return;
     }
-    // Handle bad request when unexpected "Connect: Upgrade" header is provided
-    if (/upgrade/gi.test(proxyReq.getHeader('Connection'))) {
-        handleBadRequest({ proxyReq, req, res });
-        return;
-    }
-    // Handle bad request when invalid request body is provided
-    if (hasInvalidKeys(requestBody)) {
-        handleBadRequest({ proxyReq, req, res });
-        return;
-    }
     const writeBody = (bodyData) => {
         // deepcode ignore ContentLengthInCode: bodyParser fix
         proxyReq.setHeader('Content-Length', Buffer.byteLength(bodyData));
@@ -37,12 +31,3 @@ function fixRequestBody(proxyReq, req, res) {
     }
 }
 exports.fixRequestBody = fixRequestBody;
-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,6 +1,6 @@
 {
   "name": "http-proxy-middleware",
-  "version": "2.0.8",
+  "version": "2.0.9",
   "description": "The one-liner node.js proxy middleware for connect, express and browser-sync",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",