Fixing WA crashing in Firefox private mode

Due to the way we now handle the browser cache, previously ignored errors in the Cache API were now explicitly thrown, preventing the loading of Firefox in private mode.
This commit fixes the issue and improves the stacktrace display of errors at the same time.
This commit is contained in:
David Négrier 2022-01-06 11:46:07 +01:00
parent 88509916a8
commit d713c82568
3 changed files with 13 additions and 3 deletions

View File

@ -168,6 +168,9 @@ class ConnectionManager {
} }
} catch (err) { } catch (err) {
console.error(err); console.error(err);
if (err instanceof Error) {
console.error(err.stack);
}
} }
} else { } else {
const query = urlParams.toString(); const query = urlParams.toString();

View File

@ -139,9 +139,13 @@ class LocalUserStore {
async setLastRoomUrl(roomUrl: string): Promise<void> { async setLastRoomUrl(roomUrl: string): Promise<void> {
localStorage.setItem(lastRoomUrl, roomUrl.toString()); localStorage.setItem(lastRoomUrl, roomUrl.toString());
if ("caches" in window) { if ("caches" in window) {
const cache = await caches.open(cacheAPIIndex); try {
const stringResponse = new Response(JSON.stringify({ roomUrl })); const cache = await caches.open(cacheAPIIndex);
await cache.put(`/${lastRoomUrl}`, stringResponse); const stringResponse = new Response(JSON.stringify({ roomUrl }));
await cache.put(`/${lastRoomUrl}`, stringResponse);
} catch (e) {
console.error("Could not store last room url in Browser cache. Are you using private browser mode?", e);
}
} }
} }
getLastRoomUrl(): string { getLastRoomUrl(): string {

View File

@ -78,6 +78,9 @@ export class ErrorScene extends Phaser.Scene {
*/ */
public static showError(error: unknown, scene: ScenePlugin): void { public static showError(error: unknown, scene: ScenePlugin): void {
console.error(error); console.error(error);
if (error instanceof Error) {
console.error("Stacktrace: ", error.stack);
}
console.trace(); console.trace();
if (typeof error === "string" || error instanceof String) { if (typeof error === "string" || error instanceof String) {