Fixing route() such that result.params is evaluated correctly

avoidwork · Dec 26, 2020 · 489a29a · 489a29a
1 parent dc12ace
commit 489a29a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/lib/woodland.js b/lib/woodland.js
@@ -211,11 +211,10 @@ class Woodland extends EventEmitter {
 	}
 
 	route (...args) {
-		const [req, res] = args;
+		const [req, res] = args,
+			e = err => this.emit("error", req, res, err);
 		let method = req.method === "HEAD" ? "GET" : req.method;
 
-		const e = err => this.emit("error", req, res, err);
-
 		this.emit("connect", req, res);
 		res.on("finish", () => this.emit("finish", req, res));
 
@@ -228,7 +227,7 @@ class Woodland extends EventEmitter {
 		} else if (req.allow.includes(method)) {
 			const result = this.routes(req.parsed.pathname, method);
 
-			if (result.params !== void 0) {
+			if (result.params) {
 				params(req, result.pos);
 			}
 

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