Fixing what triggers a cache update such that the middleware might mi…

…ss the first time, fixing cached `timestamp` over the lifespan of the `cache` item

README.md

@@ -42,7 +42,10 @@ Creates a hash of `arg`, used by `create()`
 Middleware to be used by an http framework
 
 ##### register (url, state)
-Registers a state in the cache
+Adds `url` to the `cache`
+
+##### unregister (url)
+Removes `url` from the `cache`
 
 ## License
 Copyright (c) 2016 Jason Mulligan

lib/etag.js

@@ -9,7 +9,8 @@ const lru = require("tiny-lru"),
 		implicitGet: /^(HEAD|GET|OPTIONS)$/,
 		explicitGet: /^GET$/i,
 		etag: /ETag:\s/i,
-		invalid: /^(a|cache|connection|content-(d|e|l|m|r)|d|ex|l|p|r|s|t|u|v|w|x)/
+		invalid: /^(a|cache|connection|content-(d|e|l|m|r)|d|ex|l|p|r|s|t|u|v|w|x)/,
+		valid: /^(200|304)$/
 	};
 
 function clone (arg) {
@@ -45,7 +46,7 @@ class ETag {
 				headers;
 
 			res.on("finish", () => {
-				if (implicit && res.statusCode === 200 && regex.etag.test(res._header)) {
+				if (implicit && regex.valid.test(res.statusCode) && regex.etag.test(res._header)) {
 					let lheaders = {};
 
 					trim(res._header).split(/\r\n/).forEach(row => {
@@ -59,7 +60,7 @@ class ETag {
 					obj.register(parsed.href, {
 						etag: lheaders.etag,
 						headers: lheaders,
-						timestamp: parseInt(new Date().getTime() / 1000, 10)
+						timestamp: cached ? cached.timestamp : parseInt(new Date().getTime() / 1000, 10)
 					});
 				}
 			});
@@ -95,6 +96,10 @@ class ETag {
 
 		return this;
 	}
+
+	unregister (uri) {
+		this.cache.remove(this.hash(uri));
+	}
 }
 
 module.exports = ETag;

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tiny-etag",
-  "version": "1.0.10",
+  "version": "1.0.11",
   "description": "ETag middleware",
   "main": "index.js",
   "scripts": {