Don't use the service worker for the feed (#5734)

forem · Jan 27, 2020 · e582e2a · e582e2a
1 parent a86ccff
commit e582e2a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/app/views/service_worker/index.js.erb b/app/views/service_worker/index.js.erb
@@ -100,12 +100,13 @@
         !url.href.includes('/admin') && // Don't fetch for administrate dashboard.
         !url.href.includes('/internal') && // Don't fetch for internal dashboard.
         !url.href.includes('/future') && // Skip for /future.
-        !url.href.includes('?preview=') && // Skip for /future.
+        !url.href.includes('?preview=') && // Skip for preview pages.
         !url.href.includes('/delayed_job') && // Skip for Delayed Job dashboard
         !url.href.includes('/sidekiq') && // Skip for Sidekiq dashboard
         !url.href.includes('/oauth/') && // Skip oauth apps
-        !url.href.includes('/robots.txt') && // Skip oauth apps
+        !url.href.includes('/robots.txt') && // Skip robots for web crawlers
         !url.href.includes('/rails/mailers') && // Skip for mailers previews in development mode
+        !url.href.includes('/feed') && // Skip the RSS feed
         caches.match('/shell_top') && // Ensure shell_top is in the cache.
         caches.match('/shell_bottom')) { // Ensure shell_bottom is in the cache.
         event.respondWith(createPageStream(event.request)); // Respond with the stream