Skip to content

Commit 560a5ae

Browse files
authored
Merge pull request gbowne1#45 from shishir-21/fix/service-worker-api-cache
Fix: exclude API routes from service worker cache
2 parents 427441a + 2d5065b commit 560a5ae

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

public/service-worker.js

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,14 @@ self.addEventListener("activate", (event) => {
5050

5151
// Fetch event handler with cache-first strategy
5252
self.addEventListener("fetch", (event) => {
53+
if (!event.request.url.startsWith("http")) return;
54+
const url = new URL(event.request.url);
55+
56+
// Do not cache API requests (like/ api/streams)
57+
if (url.pathname.startsWith("/api/")) {
58+
return;
59+
}
60+
5361
event.respondWith(
5462
caches.match(event.request).then((response) => {
5563
if (response) {
@@ -67,11 +75,11 @@ self.addEventListener("fetch", (event) => {
6775
);
6876
return response;
6977
}
70-
78+
7179
// Handle non-cached requests
7280
return fetch(event.request).catch(error => {
7381
console.error("Network request failed:", error);
74-
82+
7583
// Serve offline fallback for navigation requests
7684
if (event.request.mode === 'navigate') {
7785
return caches.match('/offline.html');

0 commit comments

Comments
 (0)