salman/igny8
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test auth store

Browse Source
This commit is contained in:
IGNY8 VPS (Salman)
2025-12-10 11:33:05 +00:00
parent 69363b9b31
commit 6a056e3589
1 changed files with 19 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
frontend/src/services/api.ts
View File

@@ -88,13 +88,15 @@ const getAuthToken = (): string | null => {
} }
// Fallback to localStorage (for cases where store hasn't initialized yet) // Fallback to localStorage (for cases where store hasn't initialized yet)
const authStorage = localStorage.getItem('auth-store'); // CRITICAL: Use 'auth-storage' to match authStore persist config
const authStorage = localStorage.getItem('auth-storage');
if (authStorage) { if (authStorage) {
const parsed = JSON.parse(authStorage); const parsed = JSON.parse(authStorage);
return parsed?.state?.token || null; return parsed?.state?.token || null;
} }
} catch (e) { } catch (e) {
// Ignore parsing errors // Ignore parsing errors
console.warn('Failed to get auth token:', e);
} }
return null; return null;
}; };
@@ -109,13 +111,15 @@ const getRefreshToken = (): string | null => {
} }
// Fallback to localStorage (for cases where store hasn't initialized yet) // Fallback to localStorage (for cases where store hasn't initialized yet)
const authStorage = localStorage.getItem('auth-store'); // CRITICAL: Use 'auth-storage' to match authStore persist config
const authStorage = localStorage.getItem('auth-storage');
if (authStorage) { if (authStorage) {
const parsed = JSON.parse(authStorage); const parsed = JSON.parse(authStorage);
return parsed?.state?.refreshToken || null; return parsed?.state?.refreshToken || null;
} }
} catch (e) { } catch (e) {
// Ignore parsing errors // Ignore parsing errors
console.warn('Failed to get refresh token:', e);
} }
return null; return null;
}; };
@@ -160,16 +164,15 @@ export async function fetchAPI(endpoint: string, options?: RequestInit & { timeo
const errorData = text ? JSON.parse(text) : null; const errorData = text ? JSON.parse(text) : null;
const errorMessage = errorData?.detail || errorData?.message || errorData?.error || response.statusText; const errorMessage = errorData?.detail || errorData?.message || errorData?.error || response.statusText;
// Check if it's an authentication credentials error // Check if it's an authentication credentials error (NOT permission/plan errors)
if (errorMessage?.includes?.('Authentication credentials') || if (errorMessage?.includes?.('Authentication credentials') ||
errorMessage?.includes?.('not authenticated') || errorMessage?.includes?.('not authenticated')) {
errorMessage?.includes?.('Invalid token') ||
errorMessage?.includes?.('Token has expired')) {
// CRITICAL: Authentication token is invalid/missing - force logout // CRITICAL: Only force logout if we're actually authenticated but token is missing/invalid
// Don't logout for permission errors or plan issues
const authState = useAuthStore.getState(); const authState = useAuthStore.getState();
if (authState?.isAuthenticated || authState?.token) { if (authState?.isAuthenticated || authState?.token) {
console.warn('Authentication token invalid - forcing logout'); console.warn('Authentication credentials missing - forcing logout');
const { logout } = useAuthStore.getState(); const { logout } = useAuthStore.getState();
logout(); logout();
@@ -186,7 +189,7 @@ export async function fetchAPI(endpoint: string, options?: RequestInit & { timeo
throw err; throw err;
} }
// Not an auth error - could be permissions/plan issue // Not an auth error - could be permissions/plan issue - don't force logout
let err: any = new Error(errorMessage); let err: any = new Error(errorMessage);
err.status = 403; err.status = 403;
err.data = errorData; err.data = errorData;
@@ -235,8 +238,13 @@ export async function fetchAPI(endpoint: string, options?: RequestInit & { timeo
const refreshData = await refreshResponse.json(); const refreshData = await refreshResponse.json();
const accessToken = refreshData.data?.access || refreshData.access; const accessToken = refreshData.data?.access || refreshData.access;
if (refreshData.success && accessToken) { if (refreshData.success && accessToken) {
// Update token in store // Update token in Zustand store AND localStorage
try { try {
// Update Zustand store directly
const { setToken } = useAuthStore.getState();
setToken(accessToken);
// Also update localStorage for immediate availability
const authStorage = localStorage.getItem('auth-storage'); const authStorage = localStorage.getItem('auth-storage');
if (authStorage) { if (authStorage) {
const parsed = JSON.parse(authStorage); const parsed = JSON.parse(authStorage);
@@ -244,7 +252,7 @@ export async function fetchAPI(endpoint: string, options?: RequestInit & { timeo
localStorage.setItem('auth-storage', JSON.stringify(parsed)); localStorage.setItem('auth-storage', JSON.stringify(parsed));
} }
} catch (e) { } catch (e) {
// Ignore storage errors console.warn('Failed to update token after refresh:', e);
} }
// Retry original request with new token // Retry original request with new token