RPJosh
/
memories
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

sw: improve strategy 

Signed-off-by: Varun Patil <radialapps@gmail.com>
pull/953/head
Varun Patil 2023-11-25 11:53:34 -08:00
parent 1971c5e3ce
commit 7b4ad788aa
1 changed files with 27 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

45
src/service-worker.ts
View File

@ -10,14 +10,14 @@ const manifest = self.__WB_MANIFEST as Array<PrecacheEntry>;
// Exclude files that are not needed // Exclude files that are not needed
const filteredManifest = manifest.filter((entry) => { const filteredManifest = manifest.filter((entry) => {
return !entry.url?.match(/LICENSE\.txt(\?.*)?$/); return !/LICENSE\.txt(\?.*)?$/.test(entry.url ?? String());
}); });
precacheAndRoute(filteredManifest); precacheAndRoute(filteredManifest);
cleanupOutdatedCaches(); cleanupOutdatedCaches();
registerRoute( registerRoute(
/^.*\/apps\/memories\/api\/video\/livephoto\/.*/, /\/apps\/memories\/api\/video\/livephoto\/.*/,
new CacheFirst({ new CacheFirst({
cacheName: 'memories-livephotos', cacheName: 'memories-livephotos',
plugins: [ plugins: [
@ -29,24 +29,16 @@ registerRoute(
}), }),
); );
// Important: Using the NetworkOnly strategy and not registering // Use CacheFirst for static assets
// a route are NOT equivalent. The NetworkOnly strategy will const cachefirst = [
// strip certain headers such as HTTP-Range, which is required /\.(?:js|css|woff2?|png|jpg|jpeg|gif|svg|ico)$/i, // Static assets
// for proper playback of videos. /\/apps\/theming\/(icon|favicon|manifest)/i, // Theming
/\/avatar/i, // User avatars
];
const networkOnly = [/^.*\/api\/.*/]; // Cache static file assets
// Use network-first for memories page for initial state such as theming
registerRoute( registerRoute(
({ url }) => url.origin === self.location.origin && url.pathname.endsWith('/apps/memories/'), ({ url }) => cachefirst.some((regex) => regex.test(url.pathname)),
new NetworkFirst({
cacheName: 'memories-pages',
}),
);
// Cache pages for same-origin requests only
registerRoute(
({ url }) => url.origin === self.location.origin && !networkOnly.some((regex) => regex.test(url.href)),
new CacheFirst({ new CacheFirst({
cacheName: 'memories-pages', cacheName: 'memories-pages',
plugins: [ plugins: [
@ -58,6 +50,23 @@ registerRoute(
}), }),
); );
// Important: Using the NetworkOnly strategy and not registering
// a route are NOT equivalent. The NetworkOnly strategy will
// strip certain headers such as HTTP-Range, which is required
// for proper playback of videos.
const netonly = [
/\/(api|ocs)\//i, // API calls
/\/csrftoken/i, // CSRF token (https://github.com/pulsejet/memories/issues/835)
];
// Use NetworkFirst for HTML pages for initial state and CSRF token
registerRoute(
({ url }) => !netonly.some((regex) => regex.test(url.pathname)),
new NetworkFirst({
cacheName: 'memories-pages',
}),
);
self.addEventListener('activate', (event) => { self.addEventListener('activate', (event) => {
// Take control of all pages under this SW's scope immediately, // Take control of all pages under this SW's scope immediately,
// instead of waiting for reload/navigation. // instead of waiting for reload/navigation.