Merge pull request #180 from Kong/fix/sensitive-content-length

filter out custom content length header

Author: jackkav

.vscode/settings.json

@@ -1,5 +1,7 @@
 {
-	"editor.formatOnSave": true,
+	"editor.codeActionsOnSave": {
+		"quickfix.biome": "explicit"
+	},
 	"[javascript]": {
 		"editor.defaultFormatter": "biomejs.biome"
 	},

biome.json

@@ -1,5 +1,5 @@
 {
-	"$schema": "https://biomejs.dev/schemas/1.4.1/schema.json",
+	"$schema": "https://biomejs.dev/schemas/1.7.3/schema.json",
 	"organizeImports": {
 		"enabled": true
 	},

lib/routes/bins/create.js

@@ -40,6 +40,10 @@ module.exports = (client) => (req, res, next) => {
 	}
 
 	mock.content.size = 0;
+	// express is sensitive to mismatched content-length header
+	mock.headers = mock.headers.filter(
+		(header) => header.name.toLowerCase() !== "content-length",
+	);
 
 	validate
 		.response(mock)

lib/routes/bins/update.js

@@ -37,6 +37,10 @@ module.exports = (client) => (req, res, next) => {
 	}
 
 	mock.content.size = 0;
+	// express is sensitive to mismatched content-length header
+	mock.headers = mock.headers.filter(
+		(header) => header.name.toLowerCase() !== "content-length",
+	);
 
 	validate
 		.response(mock)

package-lock.json

@@ -1,5 +1,5 @@
 {
-	"version": "2.0.2",
+	"version": "2.0.3",
 	"name": "mockbin",
 	"description": "Test, mock, and track HTTP requests & responses between libraries, sockets and APIs",
 	"author": "Kong (https://www.konghq.com/)",