Setting content-type header within error() instead of passing it …

…to `send()` which is a coherence issue for `onsend()`
avoidwork · Jan 3, 2021 · 305f4aa · 305f4aa
1 parent 9da80f8
commit 305f4aa
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/lib/woodland.js b/lib/woodland.js
@@ -215,8 +215,7 @@ class Woodland extends EventEmitter {
 		const ev = "error";
 
 		if (res.headersSent === false) {
-			const headers = {"content-type": "text/plain; charset=utf-8"},
-				numeric = isNaN(err.message) === false,
+			const numeric = isNaN(err.message) === false,
 				status = isNaN(res.statusCode) === false && res.statusCode >= 400 ? res.statusCode : numeric ? Number(err.message) : 500,
 				output = numeric ? http.STATUS_CODES[status] : err.message;
 
@@ -230,7 +229,8 @@ class Woodland extends EventEmitter {
 				}
 			}
 
-			res.send(output, status, headers);
+			res.header("content-type", "text/plain; charset=utf-8");
+			res.send(output, status);
 		}
 
 		if (this.listenerCount(ev) > 0) {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "woodland",
-  "version": "16.0.2",
+  "version": "16.0.3",
   "description": "Lightweight HTTP router with automatic headers",
   "main": "index.js",
   "scripts": {