fix(ui): Fix Safari sign-in bugs (#7877)

Co-authored-by: Nikos Douvlis <nikosdouvlis@gmail.com>

Author: Ephem

.changeset/yummy-rats-cross.md

@@ -0,0 +1,5 @@
+---
+'@clerk/ui': patch
+---
+
+Fix Safari sign-in redirect not completing when using OTP or impersonation.

packages/ui/src/contexts/components/SessionTasks.ts

@@ -1,4 +1,5 @@
 import { buildTaskUrl, getTaskEndpoint } from '@clerk/shared/internal/clerk-js/sessionTasks';
+import { windowNavigate } from '@clerk/shared/internal/clerk-js/windowNavigate';
 import { useClerk } from '@clerk/shared/react';
 import type { DecorateUrl, SessionResource } from '@clerk/shared/types';
 import { createContext, useContext } from 'react';
@@ -43,7 +44,7 @@ export const useSessionTasksContext = (): SessionTasksContextType => {
       // If decorateUrl modified the URL (Safari ITP fix), do a full page navigation
       // The touch endpoint URL will be an absolute URL starting with http:// or https://
       if (decoratedUrl !== redirectUrlComplete && /^https?:\/\//.test(decoratedUrl)) {
-        window.location.href = decoratedUrl;
+        windowNavigate(decoratedUrl);
         return;
       }
 

packages/ui/src/contexts/components/SignIn.ts

@@ -2,6 +2,7 @@ import { SIGN_IN_INITIAL_VALUE_KEYS } from '@clerk/shared/internal/clerk-js/cons
 import { RedirectUrls } from '@clerk/shared/internal/clerk-js/redirectUrls';
 import { getTaskEndpoint } from '@clerk/shared/internal/clerk-js/sessionTasks';
 import { buildURL } from '@clerk/shared/internal/clerk-js/url';
+import { windowNavigate } from '@clerk/shared/internal/clerk-js/windowNavigate';
 import { useClerk } from '@clerk/shared/react';
 import type { DecorateUrl, SessionResource } from '@clerk/shared/types';
 import { isAbsoluteUrl } from '@clerk/shared/url';
@@ -140,7 +141,7 @@ export const useSignInContext = (): SignInContextType => {
       // If decorateUrl modified the URL (Safari ITP fix), do a full page navigation
       // The touch endpoint URL will be an absolute URL starting with http:// or https://
       if (decoratedUrl !== redirectUrl && /^https?:\/\//.test(decoratedUrl)) {
-        window.location.href = decoratedUrl;
+        windowNavigate(decoratedUrl);
         return;
       }
 

packages/ui/src/contexts/components/SignUp.ts

@@ -2,6 +2,7 @@ import { SIGN_UP_INITIAL_VALUE_KEYS } from '@clerk/shared/internal/clerk-js/cons
 import { RedirectUrls } from '@clerk/shared/internal/clerk-js/redirectUrls';
 import { getTaskEndpoint } from '@clerk/shared/internal/clerk-js/sessionTasks';
 import { buildURL } from '@clerk/shared/internal/clerk-js/url';
+import { windowNavigate } from '@clerk/shared/internal/clerk-js/windowNavigate';
 import { useClerk } from '@clerk/shared/react';
 import type { DecorateUrl, SessionResource } from '@clerk/shared/types';
 import { isAbsoluteUrl } from '@clerk/shared/url';
@@ -135,7 +136,7 @@ export const useSignUpContext = (): SignUpContextType => {
       // If decorateUrl modified the URL (Safari ITP fix), do a full page navigation
       // The touch endpoint URL will be an absolute URL starting with http:// or https://
       if (decoratedUrl !== redirectUrl && /^https?:\/\//.test(decoratedUrl)) {
-        window.location.href = decoratedUrl;
+        windowNavigate(decoratedUrl);
         return;
       }