@abejarano/ts-express-server 1.7.1 → 1.7.2

package/dist/adapters/BunAdapter.js

@@ -339,12 +339,19 @@ const createJsonBodyParser = (app) => {
         if (!req.raw || req.body !== undefined) {
             return next();
         }
+        const method = String(req.method || "").toUpperCase();
+        if (method === "GET" || method === "HEAD") {
+            return next();
+        }
         const contentType = String(req.headers["content-type"] || "");
         if (!contentType.includes("application/json")) {
             return next();
         }
         const limit = getBodyLimit(app);
         const contentLength = parseContentLength(req.headers["content-length"]);
+        if (contentLength === 0) {
+            return next();
+        }
         if (contentLength !== undefined && contentLength > limit) {
             res.status(413).json({ message: "Payload too large" });
             return;

package/dist/adapters/ExpressAdapter.js

@@ -26,7 +26,13 @@ class ExpressAdapter {
         const express = getExpressModule();
         const bodyParser = require("body-parser");
         const cookieParser = require("cookie-parser");
-        expressApp.use(express.json());
+        const jsonParser = express.json();
+        expressApp.use((req, res, next) => {
+            if (!shouldParseJson(req)) {
+                return next();
+            }
+            return jsonParser(req, res, next);
+        });
         expressApp.use(bodyParser.urlencoded({ extended: true }));
         expressApp.use(cookieParser());
         expressApp.set("port", port);
@@ -38,3 +44,22 @@ class ExpressAdapter {
     }
 }
 exports.ExpressAdapter = ExpressAdapter;
+const shouldParseJson = (req) => {
+    const method = String(req.method || "").toUpperCase();
+    if (method === "GET" || method === "HEAD") {
+        return false;
+    }
+    const lengthHeader = req.headers["content-length"];
+    if (lengthHeader === undefined) {
+        return true;
+    }
+    const value = Array.isArray(lengthHeader) ? lengthHeader[0] : lengthHeader;
+    if (!value) {
+        return true;
+    }
+    const parsed = Number.parseInt(value, 10);
+    if (Number.isNaN(parsed)) {
+        return true;
+    }
+    return parsed > 0;
+};

package/dist/modules/CorsModule.js

@@ -44,7 +44,9 @@ const createCorsMiddleware = (options) => {
             res.set("access-control-allow-credentials", "true");
         }
         if (options.methods) {
-            res.set("access-control-allow-methods", Array.isArray(options.methods) ? options.methods.join(",") : options.methods);
+            res.set("access-control-allow-methods", Array.isArray(options.methods)
+                ? options.methods.join(",")
+                : options.methods);
         }
         if (options.allowedHeaders) {
             res.set("access-control-allow-headers", Array.isArray(options.allowedHeaders)

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@abejarano/ts-express-server",
   "author": "angel bejarano / angel.bejarano@jaspesoft.com",
-  "version": "1.7.1",
+  "version": "1.7.2",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "files": [