Remove the hash fragment the URL when determining if there's a match. (…

…#291)

lib/functions.js

@@ -23,6 +23,8 @@ module.exports = {
   stripIgnoredUrlParameters: function(originalUrl,
     ignoreUrlParametersMatching) {
     var url = new URL(originalUrl);
+    // Remove the hash; see https://github.com/GoogleChrome/sw-precache/issues/290
+    url.hash = '';
 
     url.search = url.search.slice(1) // Exclude initial '?'
       .split('&') // Split into an array of 'key=value' strings

service-worker.tmpl

@@ -133,8 +133,8 @@ self.addEventListener('fetch', function(event) {
     // handlers a chance to handle the request if need be.
     var shouldRespond;
 
-    // First, remove all the ignored parameter and see if we have that URL
-    // in our cache. If so, great! shouldRespond will be true.
+    // First, remove all the ignored parameters and hash fragment, and see if we
+    // have that URL in our cache. If so, great! shouldRespond will be true.
     var url = stripIgnoredUrlParameters(event.request.url, ignoreUrlParametersMatching);
     shouldRespond = urlsToCacheKeys.has(url);
 

test/test.js

@@ -332,9 +332,9 @@ describe('stripIgnoredUrlParameters', function() {
     done();
   });
 
-  it('should work when there\'s a hash fragment', function(done) {
+  it('should remove the hash fragment', function(done) {
     var strippedUrl = externalFunctions.stripIgnoredUrlParameters(testUrl + '#hash', [/^t/, /^f/]);
-    assert.strictEqual(strippedUrl, 'http://example.com/index.html?one=1#hash');
+    assert.strictEqual(strippedUrl, 'http://example.com/index.html?one=1');
     done();
   });
 });