Auth: Prevent /start calls when logged in

Crossmint · Feb 14, 2025 · 9eeed22 · 9eeed22
1 parent d31f5c8
commit 9eeed22
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/packages/client/ui/react-ui/src/providers/CrossmintAuthProvider.tsx b/packages/client/ui/react-ui/src/providers/CrossmintAuthProvider.tsx
@@ -173,6 +173,7 @@ export function CrossmintAuthProvider({
                     appearance={appearance}
                 >
                     <AuthFormProvider
+                        preFetchOAuthUrls={getAuthStatus() === "logged-out"}
                         initialState={{
                             appearance,
                             setDialogOpen,

diff --git a/packages/client/ui/react-ui/src/providers/auth/AuthFormProvider.tsx b/packages/client/ui/react-ui/src/providers/auth/AuthFormProvider.tsx
@@ -37,6 +37,12 @@ type ContextInitialStateProps = {
     embeddedWallets: CrossmintAuthWalletConfig;
 };
 
+type AuthFormProviderProps = {
+    preFetchOAuthUrls: boolean;
+    initialState: ContextInitialStateProps;
+    children: ReactNode;
+};
+
 const AuthFormContext = createContext<AuthFormContextType | undefined>(undefined);
 
 export const useAuthForm = () => {
@@ -47,10 +53,7 @@ export const useAuthForm = () => {
     return context;
 };
 
-export const AuthFormProvider = ({
-    children,
-    initialState,
-}: { children: ReactNode; initialState: ContextInitialStateProps }) => {
+export const AuthFormProvider = ({ preFetchOAuthUrls, initialState, children }: AuthFormProviderProps) => {
     const { crossmintAuth } = useCrossmintAuth();
     const [step, setStep] = useState<AuthStep>("initial");
     const [error, setError] = useState<string | null>(null);
@@ -90,8 +93,11 @@ export const AuthFormProvider = ({
     }, [loginMethods, crossmintAuth]);
 
     useEffect(() => {
-        preFetchAndSetOauthUrl();
-    }, [preFetchAndSetOauthUrl]);
+        // Only pre-fetch oauth urls if the user is not logged in
+        if (preFetchOAuthUrls) {
+            preFetchAndSetOauthUrl();
+        }
+    }, [preFetchAndSetOauthUrl, preFetchOAuthUrls]);
 
     const handleToggleDialog = (open: boolean) => {
         setDialogOpen?.(open);