fix(auth): remove redundant token storage write

This commit is contained in:
너이름 2026-05-11 06:25:40 +09:00
parent c8374e6c5b
commit 935cd7b7c9
2 changed files with 2 additions and 10 deletions

View file

@ -8,11 +8,7 @@ export default function AuthCallbackPage() {
// TokenHandler uses useGlobalLoadingEffect to show the loading screen
return (
<Suspense fallback={null}>
<TokenHandler
redirectPath="/dashboard"
tokenParamName="token"
storageKey="surfsense_bearer_token"
/>
<TokenHandler redirectPath="/dashboard" tokenParamName="token" />
</Suspense>
);
}

View file

@ -9,7 +9,6 @@ import { trackLoginSuccess } from "@/lib/posthog/events";
interface TokenHandlerProps {
redirectPath?: string; // Default path to redirect after storing token (if no saved path)
tokenParamName?: string; // Name of the URL parameter containing the token
storageKey?: string; // Key to use when storing in localStorage (kept for backwards compatibility)
}
/**
@ -19,12 +18,10 @@ interface TokenHandlerProps {
*
* @param redirectPath - Default path to redirect after storing token (default: '/dashboard')
* @param tokenParamName - Name of the URL parameter containing the token (default: 'token')
* @param storageKey - Key to use when storing in localStorage (default: 'surfsense_bearer_token')
*/
const TokenHandler = ({
redirectPath = "/dashboard",
tokenParamName = "token",
storageKey = "surfsense_bearer_token",
}: TokenHandlerProps) => {
// Always show loading for this component - spinner animation won't reset
useGlobalLoadingEffect(true);
@ -45,7 +42,6 @@ const TokenHandler = ({
}
sessionStorage.removeItem("login_success_tracked");
localStorage.setItem(storageKey, token);
setBearerToken(token);
if (refreshToken) {
@ -78,7 +74,7 @@ const TokenHandler = ({
};
run();
}, [tokenParamName, storageKey, redirectPath]);
}, [tokenParamName, redirectPath]);
// Return null - the global provider handles the loading UI
return null;