From 3b74410a7df243afb5eeb99f56da1cf00f5986b9 Mon Sep 17 00:00:00 2001 From: erdemcaygor Date: Wed, 30 Jul 2025 17:18:57 +0300 Subject: [PATCH] refactoring --- .../src/lib/services/browser-token-storage.service.ts | 7 +++---- .../src/lib/strategies/auth-code-flow-strategy.ts | 10 ---------- 2 files changed, 3 insertions(+), 14 deletions(-) diff --git a/npm/ng-packs/packages/oauth/src/lib/services/browser-token-storage.service.ts b/npm/ng-packs/packages/oauth/src/lib/services/browser-token-storage.service.ts index 7c5b51d2e7..66b894a25a 100644 --- a/npm/ng-packs/packages/oauth/src/lib/services/browser-token-storage.service.ts +++ b/npm/ng-packs/packages/oauth/src/lib/services/browser-token-storage.service.ts @@ -14,24 +14,23 @@ export class BrowserTokenStorageService implements OAuthStorage { } setItem(key: string, data: string): void { - console.log(key); this.writeCookie(key, data); } readCookie(name: string): string | null { - if (typeof document === 'undefined') return null; // SSR'de yok + if (typeof document === 'undefined') return null; const match = document.cookie.match(new RegExp('(^| )' + name + '=([^;]+)')); return match ? decodeURIComponent(match[2]) : null; } writeCookie(name: string, value: string, days = 7): void { - if (typeof document === 'undefined') return; // SSR'de yok + if (typeof document === 'undefined') return; const expires = new Date(Date.now() + days * 86400000).toUTCString(); document.cookie = `${name}=${encodeURIComponent(value)}; expires=${expires}; path=/; Secure; SameSite=Lax`; } removeCookie(name: string): void { - if (typeof document === 'undefined') return; // SSR'de yok + if (typeof document === 'undefined') return; document.cookie = `${name}=; Max-Age=0; path=/;`; } } diff --git a/npm/ng-packs/packages/oauth/src/lib/strategies/auth-code-flow-strategy.ts b/npm/ng-packs/packages/oauth/src/lib/strategies/auth-code-flow-strategy.ts index d5eb14ee24..f488bad665 100644 --- a/npm/ng-packs/packages/oauth/src/lib/strategies/auth-code-flow-strategy.ts +++ b/npm/ng-packs/packages/oauth/src/lib/strategies/auth-code-flow-strategy.ts @@ -31,13 +31,9 @@ export class AuthCodeFlowStrategy extends AuthFlowStrategy { } private checkRememberMeOption() { - console.log('checkRememberMeOption called ---->>>>>'); const accessToken = this.oAuthService.getAccessToken(); - console.log('accessToken New ----->>>>', accessToken); const isTokenExpire = isTokenExpired(this.oAuthService.getAccessTokenExpiration()); - console.log('isTokenExpire New ----->>>>', isTokenExpire); let rememberMe = this.rememberMeService.get(); - console.log('rememberMe New ----->>>>', rememberMe); if (accessToken && !rememberMe) { const rememberMeValue = this.rememberMeService.getFromToken(accessToken); @@ -82,7 +78,6 @@ export class AuthCodeFlowStrategy extends AuthFlowStrategy { } protected listenToTokenReceived() { - console.log('listenToTokenReceived'); if (isPlatformBrowser(this.platformId)) { this.oAuthService.events .pipe( @@ -98,8 +93,6 @@ export class AuthCodeFlowStrategy extends AuthFlowStrategy { } navigateToLogin(queryParams?: Params) { - console.log('navigateToLogin'); - console.log(isPlatformBrowser(this.platformId)); if (isPlatformBrowser(this.platformId)) { let additionalState = ''; if (queryParams?.returnUrl) { @@ -112,7 +105,6 @@ export class AuthCodeFlowStrategy extends AuthFlowStrategy { } checkIfInternalAuth(queryParams?: Params) { - console.log('checkIfInternalAuth'); if (isPlatformBrowser(this.platformId)) { this.oAuthService.initCodeFlow('', this.getCultureParams(queryParams)); return false; @@ -129,8 +121,6 @@ export class AuthCodeFlowStrategy extends AuthFlowStrategy { } login(queryParams?: Params) { - console.log('login'); - console.log(isPlatformBrowser(this.platformId)); if (isPlatformBrowser(this.platformId)) { this.oAuthService.initCodeFlow('', this.getCultureParams(queryParams)); return of(null);