fix(clerk-js): Replace state when merging fragment into url

clerk · Jun 6, 2023 · 18638b8 · 18638b8
1 parent 00d8d74
commit 18638b8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/packages/clerk-js/src/ui/router/PathRouter.tsx b/packages/clerk-js/src/ui/router/PathRouter.tsx
@@ -38,7 +38,8 @@ export const PathRouter = ({ basePath, preservedParams, children }: PathRouterPr
     const convertHashToPath = async () => {
       if (hasUrlInFragment(window.location.hash)) {
         const url = mergeFragmentIntoUrl(new URL(window.location.href));
-        await internalNavigate(url.href);
+        window.history.replaceState(window.history.state, '', url.href);
+        await internalNavigate(url.href); // make this navigation as well since replaceState is asynchronous
         setStripped(true);
       }
     };

diff --git a/packages/clerk-js/src/ui/router/__tests__/PathRouter.test.tsx b/packages/clerk-js/src/ui/router/__tests__/PathRouter.test.tsx
@@ -54,6 +54,7 @@ describe('PathRouter', () => {
   beforeAll(() => {
     // @ts-ignore
     delete window.location;
+    history.replaceState = jest.fn();
   });
 
   afterAll(() => {