Merge pull request authts#330 from authts/fix-redirect-unload

fix(RedirectNavigator): wait for 'unload' instead of 'beforeunload'
ncats · Jan 16, 2022 · 02733aa · 02733aa
2 parents 74ee0bc + 608668b
commit 02733aa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/navigators/RedirectNavigator.ts b/src/navigators/RedirectNavigator.ts
@@ -32,7 +32,7 @@ export class RedirectNavigator implements INavigator {
                 this._logger.create("navigate");
                 const promise = new Promise((resolve, reject) => {
                     abort = reject;
-                    window.addEventListener("beforeunload", () => resolve(null));
+                    window.addEventListener("unload", () => resolve(null));
                 });
                 redirect(params.url);
                 return await (promise as Promise<never>);

diff --git a/test/setup.ts b/test/setup.ts
@@ -3,7 +3,7 @@ import { Log } from "../src";
 beforeAll(() => {
     globalThis.fetch = jest.fn();
 
-    const unload = () => window.dispatchEvent(new Event("beforeunload"));
+    const unload = () => window.dispatchEvent(new Event("unload"));
 
     const location = Object.defineProperties({}, {
         ...Object.getOwnPropertyDescriptors(window.location),