PowerOn/frontend_nyla
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix:added more rolelabel logging to see why missing pages

Browse source
This commit is contained in:
Ida Dittrich 2026-01-05 07:12:37 +01:00
parent fc55a25f0b
commit 5f22c7be77
1 changed files with 63 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

73
src/hooks/useUsers.ts
View file

@ -30,14 +30,28 @@ export function useCurrentUser() {
// Check if we already have user data in sessionStorage cache // Check if we already have user data in sessionStorage cache
const cachedUser = getUserDataCache(); const cachedUser = getUserDataCache();
if (cachedUser && cachedUser.username) { if (cachedUser && cachedUser.username) {
// Use cached user data - permissions are checked via RBAC API, not client-side // Check if cached user has roleLabels - if empty, refetch from API
setUser(cachedUser); const hasRoleLabels = Array.isArray(cachedUser.roleLabels) && cachedUser.roleLabels.length > 0;
console.log('✅ Using cached user data from sessionStorage (persists during session):', { const hasPrivilege = !!cachedUser.privilege;
username: cachedUser.username,
roleLabels: cachedUser.roleLabels, if (!hasRoleLabels && !hasPrivilege) {
privilege: cachedUser.privilege console.warn('⚠️ Cached user data has no roleLabels or privilege, refetching from API:', {
}); username: cachedUser.username,
return; roleLabels: cachedUser.roleLabels,
privilege: cachedUser.privilege
});
// Clear cache and continue to fetch from API
clearUserDataCache();
} else {
// Use cached user data - permissions are checked via RBAC API, not client-side
setUser(cachedUser);
console.log('✅ Using cached user data from sessionStorage (persists during session):', {
username: cachedUser.username,
roleLabels: cachedUser.roleLabels,
privilege: cachedUser.privilege
});
return;
}
} }
// JWT tokens are now stored in httpOnly cookies, so we fetch user data from API // JWT tokens are now stored in httpOnly cookies, so we fetch user data from API
@ -91,12 +105,31 @@ export function useCurrentUser() {
throw new Error('Invalid user data received from API'); throw new Error('Invalid user data received from API');
} }
// Check if API returned roleLabels - if not, log warning but still cache
const hasRoleLabels = Array.isArray(data.roleLabels) && data.roleLabels.length > 0;
const hasPrivilege = !!data.privilege;
if (!hasRoleLabels && !hasPrivilege) {
console.warn('⚠️ User data from API has no roleLabels or privilege - this may cause RBAC issues:', {
username: data.username,
roleLabels: data.roleLabels,
privilege: data.privilege,
allKeys: Object.keys(data),
fullResponse: JSON.stringify(data, null, 2)
});
// Still cache it, but log the issue - backend RBAC should handle permissions
// However, if backend expects roleLabels, this will cause problems
}
// Cache user data (permissions are checked via RBAC API) // Cache user data (permissions are checked via RBAC API)
setUserDataCache(data); setUserDataCache(data);
console.log('✅ User data fetched from API and cached in sessionStorage (secure):', { console.log('✅ User data fetched from API and cached in sessionStorage (secure):', {
username: data.username, username: data.username,
roleLabels: data.roleLabels, roleLabels: data.roleLabels,
privilege: data.privilege roleLabelsLength: Array.isArray(data.roleLabels) ? data.roleLabels.length : 0,
privilege: data.privilege,
hasRoleLabels,
hasPrivilege
}); });
setUser(data); setUser(data);
} catch (error: any) { } catch (error: any) {
@ -267,9 +300,29 @@ export function useCurrentUser() {
// Try to load user from sessionStorage cache first for faster initial load // Try to load user from sessionStorage cache first for faster initial load
const cachedUser = getUserDataCache(); const cachedUser = getUserDataCache();
if (cachedUser && cachedUser.username) { if (cachedUser && cachedUser.username) {
// Check if cached user has roleLabels - if empty, refetch from API
const hasRoleLabels = Array.isArray(cachedUser.roleLabels) && cachedUser.roleLabels.length > 0;
const hasPrivilege = !!cachedUser.privilege;
if (!hasRoleLabels && !hasPrivilege) {
console.warn('⚠️ Cached user data has no roleLabels or privilege, refetching from API:', {
username: cachedUser.username,
roleLabels: cachedUser.roleLabels,
privilege: cachedUser.privilege
});
// Clear cache and refetch
clearUserDataCache();
fetchCurrentUser();
return;
}
// Use cached user data - permissions are checked via RBAC API // Use cached user data - permissions are checked via RBAC API
setUser(cachedUser); setUser(cachedUser);
console.log('✅ Using cached user data from sessionStorage on mount (persists during session)'); console.log('✅ Using cached user data from sessionStorage on mount (persists during session):', {
username: cachedUser.username,
roleLabels: cachedUser.roleLabels,
privilege: cachedUser.privilege
});
} }
// For OAuth authentication, wait a bit longer before fetching user data // For OAuth authentication, wait a bit longer before fetching user data