From 327cc8b09c13204db5e177297111bef2954e01d4 Mon Sep 17 00:00:00 2001 From: Juan Escalada Date: Wed, 13 May 2026 11:41:40 +0900 Subject: [PATCH] fix: remove timeout for deleting lastOnline store Signed-off-by: Juan Escalada --- .../src/components/AutoLogout/AutoLogout.tsx | 13 ------------- .../components/AutoLogout/disconnectedUsers.test.ts | 2 ++ 2 files changed, 2 insertions(+), 13 deletions(-) diff --git a/packages/core-components/src/components/AutoLogout/AutoLogout.tsx b/packages/core-components/src/components/AutoLogout/AutoLogout.tsx index 47d13d4605..098215ba5f 100644 --- a/packages/core-components/src/components/AutoLogout/AutoLogout.tsx +++ b/packages/core-components/src/components/AutoLogout/AutoLogout.tsx @@ -231,27 +231,15 @@ const parseConfig = ( export const AutoLogout = (props: AutoLogoutProps): JSX.Element | null => { const identityApi = useApi(identityApiRef); const configApi = useApi(configApiRef); - const isLoggedRef = useRef(null); const [isLogged, setIsLogged] = useState(null); const lastSeenOnlineStore: TimestampStore = useMemo( () => new DefaultTimestampStore(LAST_SEEN_ONLINE_STORAGE_KEY), [], ); - useEffect(() => { - isLoggedRef.current = isLogged; - }, [isLogged]); - useEffect(() => { let cancelled = false; - const timeoutId = setTimeout(() => { - if (cancelled) return; - if (isLoggedRef.current === null || isLoggedRef.current === false) { - lastSeenOnlineStore.delete(); - } - }, 3000); - async function checkLogin(identity: IdentityApi) { try { const creds = await identity.getCredentials(); @@ -272,7 +260,6 @@ export const AutoLogout = (props: AutoLogoutProps): JSX.Element | null => { return () => { cancelled = true; - clearTimeout(timeoutId); }; }, [lastSeenOnlineStore, identityApi]); diff --git a/packages/core-components/src/components/AutoLogout/disconnectedUsers.test.ts b/packages/core-components/src/components/AutoLogout/disconnectedUsers.test.ts index 80da5bca93..090cd197d9 100644 --- a/packages/core-components/src/components/AutoLogout/disconnectedUsers.test.ts +++ b/packages/core-components/src/components/AutoLogout/disconnectedUsers.test.ts @@ -48,6 +48,8 @@ describe('useLogoutDisconnectedUserEffect', () => { renderHook(() => useLogoutDisconnectedUserEffect(props)); expect(mockTimestampStore.get).not.toHaveBeenCalled(); + expect(mockTimestampStore.delete).not.toHaveBeenCalled(); + expect(mockTimestampStore.save).not.toHaveBeenCalled(); expect(mockIdentityApi.signOut).not.toHaveBeenCalled(); });