helix-admin/helix-engage
1
0
Fork 0
mirror of https://dev.azure.com/globalhealthx/EMR/_git/helix-engage synced 2026-04-11 18:28:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: await logout before navigating, prevent cancelled fetch

Browse Source 
- Logout is now async — awaits sidecar /auth/logout before clearing tokens
- confirmSignOut awaits logout() before navigate('/login')
- 5 second timeout on logout fetch to prevent indefinite hang
- Added console.warn on logout failure

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
saridsa2
2026-03-24 15:22:39 +05:30
parent d4f33d6c06
commit 13e81ba9fb
2 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/components/layout/sidebar.tsx
View File

@@ -132,9 +132,9 @@ export const Sidebar = ({ activeUrl = "/" }: SidebarProps) => {
setLogoutOpen(true); setLogoutOpen(true);
}; };
const confirmSignOut = () => { const confirmSignOut = async () => {
setLogoutOpen(false); setLogoutOpen(false);
logout(); await logout();
navigate('/login'); navigate('/login');
}; };

17
src/providers/auth-provider.tsx
View File

@@ -95,15 +95,20 @@ export const AuthProvider = ({ children }: AuthProviderProps) => {
setIsAuthenticated(true); setIsAuthenticated(true);
}, []); }, []);
const logout = useCallback(() => { const logout = useCallback(async () => {
// Notify sidecar to unlock Redis session + Ozonetel logout // Notify sidecar to unlock Redis session + Ozonetel logout — await before clearing tokens
const token = localStorage.getItem('helix_access_token'); const token = localStorage.getItem('helix_access_token');
if (token) { if (token) {
const apiUrl = import.meta.env.VITE_API_URL ?? 'http://localhost:4100'; const apiUrl = import.meta.env.VITE_API_URL ?? 'http://localhost:4100';
fetch(`${apiUrl}/auth/logout`, { try {
method: 'POST', await fetch(`${apiUrl}/auth/logout`, {
headers: { Authorization: `Bearer ${token}` }, method: 'POST',
}).catch(() => {}); headers: { Authorization: `Bearer ${token}` },
signal: AbortSignal.timeout(5000),
});
} catch (err) {
console.warn('Logout cleanup failed:', err);
}
} }
setUser(DEFAULT_USER); setUser(DEFAULT_USER);