Setting res.statusCode from error(), conditionally setting `x-res…

…ponse-time` response header from `res.send()`
avoidwork · Jan 3, 2021 · eb9c9db · eb9c9db
1 parent d074291
commit eb9c9db
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/lib/woodland.js b/lib/woodland.js
@@ -128,8 +128,8 @@ class Woodland extends EventEmitter {
 			if (res.headersSent === false) {
 				[body, status, headers] = this.onsend(req, res, body, status, headers);
 
-				if (this.time) {
-					headers["x-response-time"] = `${ms(req.precise.stop().diff(), this.digit)}`;
+				if (this.time && res.getHeader("x-response-time") === void 0) {
+					res.header("x-response-time", `${ms(req.precise.stop().diff(), this.digit)}`);
 				}
 
 				if (pipeable(req.method, body)) {
@@ -230,6 +230,7 @@ class Woodland extends EventEmitter {
 			}
 
 			res.header("content-type", "text/plain; charset=utf-8");
+			res.statusCode = status;
 			res.send(output, status);
 		}
 

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.3",
+  "version": "16.0.4",
   "description": "Lightweight HTTP router with automatic headers",
   "main": "index.js",
   "scripts": {