Fixing router.onsend() override such that 204/304 responses will no…

…t have a response body set from the renderers
avoidwork · Jan 4, 2021 · 37d4452 · 37d4452
1 parent 15fc7d4
commit 37d4452
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/lib/utility.js b/lib/utility.js
@@ -419,8 +419,8 @@ function bootstrap (obj) {
 
 	// Setting up router
 	obj.router.addListener("connect", obj.connect.bind(obj));
-	obj.router.onsend = (req, res, body, status, headers) => {
-		if (body === null || typeof body.on !== "function") {
+	obj.router.onsend = (req, res, body = "", status = 200, headers) => {
+		if (status !== 204 && status !== 304 && (body === null || typeof body.on !== "function")) {
 			body = obj.render(req, res, obj.final(req, res, hypermedia(req, res, serialize(req, res, body)))); // eslint-disable-line no-use-before-define
 		}
 

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "tenso",
   "description": "Tenso is an HTTP REST API framework",
-  "version": "16.0.4",
+  "version": "16.0.5",
   "homepage": "https://github.com/avoidwork/tenso",
   "author": "Jason Mulligan <[email protected]>",
   "repository": {