Updating LRU cache & updating unregister() to call cache.delete()…

…, refactoring `valid()` to use `String.includes()`
avoidwork · Jan 13, 2019 · c217b4d · c217b4d
1 parent cbf8581
commit c217b4d
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/lib/etag.js b/lib/etag.js
@@ -78,13 +78,13 @@ class ETag {
 	}
 
 	unregister (key) {
-		this.cache.remove(key);
+		this.cache.delete(key);
 	}
 
 	valid (headers) {
 		const header = headers["cache-control"] || "";
 
-		return header === "" || (header.indexOf("no-cache") === -1 && header.indexOf("no-store") === -1);
+		return header.length === 0 || (header.includes("no-cache") === false && header.includes("no-store") === false); // eslint-disable-line no-extra-parens
 	}
 }
 

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "tiny-etag",
-  "version": "2.0.9",
+  "version": "2.0.10",
   "description": "ETag middleware",
   "main": "index.js",
   "scripts": {
@@ -29,7 +29,7 @@
   "homepage": "http://avoidwork.github.io/tiny-etag/",
   "dependencies": {
     "murmurhash3js": "^3.0.1",
-    "tiny-lru": "^5.0.4"
+    "tiny-lru": "^5.1.0"
   },
   "devDependencies": {
     "grunt": "^1.0.3",