Upgrading LRU, simplifying blacklist()

avoidwork · Mar 29, 2018 · 2ff1021 · 2ff1021
1 parent 2489b98
commit 2ff1021
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/lib/woodland.js b/lib/woodland.js
@@ -47,9 +47,7 @@ class Woodland {
 	}
 
 	blacklist (fn) {
-		if (this.blacklisted.has(fn.hash) === false) {
-			this.blacklisted.add(fn.hash);
-		}
+		this.blacklisted.add(fn.hash);
 
 		return this;
 	}
@@ -356,7 +354,7 @@ class Woodland {
 	}
 
 	routes (uri, method, override = false) {
-		const key = this.hash(method + delimiter + uri),
+		const key = `${method}${delimiter}${uri}`,
 			cached = override === false ? this.cache.get(key) : void 0;
 		let result;
 

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": "4.3.9",
+  "version": "4.3.10",
   "description": "Lightweight HTTP/HTTPS/HTTP2 router with automatic `Allow` & `CORS` headers",
   "main": "index.js",
   "scripts": {
@@ -33,7 +33,7 @@
   "dependencies": {
     "murmurhash3js": "^3.0.1",
     "tiny-coerce": "^1.0.1",
-    "tiny-lru": "^1.5.1",
+    "tiny-lru": "^1.5.2",
     "tiny-parse": "^2.1.2"
   },
   "devDependencies": {