Altitude/AltitudeWeb
5
0
Fork 0
Code Issues 12 Pull Requests Actions Packages Projects Releases Wiki Activity

Remove redundant debug logging in auth.service.ts and add logging in hasAccess method for authorization checks.

Browse Source
This commit is contained in:
akastijn 2025-08-02 21:55:39 +02:00
parent 2b96957876
commit 15c3cc7f26
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
frontend/src/app/services/auth.service.ts
View File

@ -64,7 +64,6 @@ export class AuthService {
} }
const claims = this.extractJwtClaims(jwt); const claims = this.extractJwtClaims(jwt);
console.log("Logged in with ", claims);
this.userClaimsSubject.next(claims); this.userClaimsSubject.next(claims);
this.isAuthenticatedSubject.next(true); this.isAuthenticatedSubject.next(true);
return true; return true;
@ -89,7 +88,6 @@ export class AuthService {
localStorage.setItem('jwt', jwt); localStorage.setItem('jwt', jwt);
const claims = this.extractJwtClaims(jwt); const claims = this.extractJwtClaims(jwt);
console.log("Logged in with ", claims);
this.userClaimsSubject.next(claims); this.userClaimsSubject.next(claims);
} }
@ -110,6 +108,7 @@ export class AuthService {
public hasAccess(requiredAuthorizations: string[]): boolean { public hasAccess(requiredAuthorizations: string[]): boolean {
const userAuthorizations = this.getUserAuthorizations(); const userAuthorizations = this.getUserAuthorizations();
console.log(requiredAuthorizations, userAuthorizations, requiredAuthorizations.some(auth => userAuthorizations.includes(auth)));
return requiredAuthorizations.some(auth => userAuthorizations.includes(auth)); return requiredAuthorizations.some(auth => userAuthorizations.includes(auth));
} }
} }