Zef Hemel
0627e7cb49
The service worker's proxy fallback only matched the wrapped "Offline" error from HttpSpacePrimitives, but raw fetch() throws browser-native errors like "Failed to fetch" (Chrome), "NetworkError..." (Firefox), or "Load failed" (Safari). This caused a hard 503 instead of falling through to serve local data. Additionally, the `fullSyncConfirmed` flag reset on every service worker restart, forcing unnecessary proxying even when synced data existed in IndexedDB. In this commit: - Use `isNetworkError()` in proxy_router.ts to catch all browser network errors and fall through to local serving instead of returning 503 * Recover `fullSyncConfirmed` from persisted sync snapshot on SW restart, so previously synced spaces serve locally immediately * Adds e2e tests trying to emulate this behavior and prevent regressions
31 lines
1.2 KiB
TypeScript
31 lines
1.2 KiB
TypeScript
export const maximumDocumentSize: number = 10; // MiB
|
|
export const defaultLinkStyle: string = "wikilink";
|
|
export const offlineError: Error = new Error("Offline");
|
|
export const notFoundError: Error = new Error("Not found");
|
|
export const notAuthenticatedError: Error = new Error("Unauthenticated");
|
|
export const wrongSpacePathError: Error = new Error(
|
|
"Space folder path different on server, reloading the page",
|
|
);
|
|
export const pingTimeout: number = 2000;
|
|
export const pingInterval: number = 5000;
|
|
|
|
/**
|
|
* Determines if an error thrown by fetch() is a network-level error
|
|
* (as opposed to a server-side or application error).
|
|
*
|
|
* Browser-specific error messages when there is no internet connection:
|
|
* - Firefox: "NetworkError when attempting to fetch resource"
|
|
* - Safari (service worker): "FetchEvent.respondWith received an error: TypeError: Load failed"
|
|
* - Safari (no service worker): "Load failed"
|
|
* - Chrome: "Failed to fetch"
|
|
*/
|
|
export function isNetworkError(e: any): boolean {
|
|
const msg = (e?.message || "").toLowerCase();
|
|
return (
|
|
msg.includes("fetch") ||
|
|
msg.includes("load failed") ||
|
|
msg.includes("networkerror") ||
|
|
msg.includes("unavailable")
|
|
);
|
|
}
|