paulmataruso/patchmon.net
1
0
Fork 0
mirror of https://github.com/9technologygroup/patchmon.net.git synced 2025-11-04 14:03:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(frontend): don't store permissions in localstorage

Browse Source
This commit is contained in:
tigattack
2025-09-27 02:14:36 +01:00
parent 102546e45d
commit 175042690e
2 changed files with 2 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
frontend/src/contexts/AuthContext.jsx
View File

@@ -41,7 +41,6 @@ export const AuthProvider = ({ children }) => {
if (response.ok) { if (response.ok) {
const data = await response.json(); const data = await response.json();
setPermissions(data); setPermissions(data);
localStorage.setItem("permissions", JSON.stringify(data));
return data; return data;
} else { } else {
console.error("Failed to fetch permissions"); console.error("Failed to fetch permissions");
@@ -67,25 +66,19 @@ export const AuthProvider = ({ children }) => {
useEffect(() => { useEffect(() => {
const storedToken = localStorage.getItem("token"); const storedToken = localStorage.getItem("token");
const storedUser = localStorage.getItem("user"); const storedUser = localStorage.getItem("user");
const storedPermissions = localStorage.getItem("permissions");
if (storedToken && storedUser) { if (storedToken && storedUser) {
try { try {
setToken(storedToken); setToken(storedToken);
setUser(JSON.parse(storedUser)); setUser(JSON.parse(storedUser));
if (storedPermissions) { // Fetch permissions from backend
setPermissions(JSON.parse(storedPermissions));
} else {
// Use the proper fetchPermissions function
fetchPermissions(storedToken); fetchPermissions(storedToken);
}
// User is authenticated, skip setup check // User is authenticated, skip setup check
setAuthPhase(AUTH_PHASES.READY); setAuthPhase(AUTH_PHASES.READY);
} catch (error) { } catch (error) {
console.error("Error parsing stored user data:", error); console.error("Error parsing stored user data:", error);
localStorage.removeItem("token"); localStorage.removeItem("token");
localStorage.removeItem("user"); localStorage.removeItem("user");
localStorage.removeItem("permissions");
// Move to setup check phase // Move to setup check phase
setAuthPhase(AUTH_PHASES.CHECKING_SETUP); setAuthPhase(AUTH_PHASES.CHECKING_SETUP);
} }
@@ -153,7 +146,6 @@ export const AuthProvider = ({ children }) => {
setPermissions(null); setPermissions(null);
localStorage.removeItem("token"); localStorage.removeItem("token");
localStorage.removeItem("user"); localStorage.removeItem("user");
localStorage.removeItem("permissions");
} }
}; };

1
frontend/src/utils/api.js
View File

@@ -39,7 +39,6 @@ api.interceptors.response.use(
// Handle unauthorized // Handle unauthorized
localStorage.removeItem("token"); localStorage.removeItem("token");
localStorage.removeItem("user"); localStorage.removeItem("user");
localStorage.removeItem("permissions");
window.location.href = "/login"; window.location.href = "/login";
} }
} }