Small improvements

1. Adding debugger for non production environments for oidc client.
2. Improve the timing when the UserManager is triggered to reload the session.
3. Suppress false-positive error logging after deleted IFrame

.github/workflows/build.yml

@@ -53,7 +53,7 @@ jobs:
         uses: cloudflare/[email protected]
         with:
           apiToken: ${{ secrets.CF_API_TOKEN }}
-          command: pages publish --project-name=demo-engity-green dist/app-green
+          command: pages deploy --project-name=demo-engity-green dist/app-green
 
   deploy-red:
     name: "Deploy: red"
@@ -75,5 +75,5 @@ jobs:
         uses: cloudflare/[email protected]
         with:
           apiToken: ${{ secrets.CF_API_TOKEN }}
-          command: pages publish --project-name=demo-engity-red dist/app-red
+          command: pages deploy --project-name=demo-engity-red dist/app-red
 

src/app/app.module.ts

@@ -21,16 +21,20 @@ import { AppTranslateLoader } from './core/i18n';
 
 class NgErrorHandler implements ErrorHandler {
     handleError(error) {
+        let foo = 'bar';
         if (
             unloaded &&
-            'rejection' in error &&
-            error.rejection &&
-            'message' in error.rejection &&
-            error.rejection.message === 'Failed to fetch'
+            ((!error?.rejection && error?.message === 'Failed to fetch') ||
+                error?.rejection?.message === 'Failed to fetch')
         ) {
             // Fetch error after unloading the window could happen is will be silently ignored.
         } else {
-            console.error(error);
+            console.error(error, {
+                unloaded: unloaded,
+                rejection: error?.rejection,
+                message: error?.rejection?.message,
+                foo: foo,
+            });
         }
     }
 }

src/app/core/components/after-login.component.ts

@@ -31,6 +31,10 @@ export class AfterLoginComponent extends BasePageComponent implements OnInit {
 
     async ngOnInit() {
         await super.ngOnInit();
+        if (!this.silent) {
+            await this._authService.initialize();
+        }
+
         const variant = await firstValueFrom(this.variant);
         try {
             await this._authService.loginCallback(this.silent);

src/app/core/components/after-logout.component.ts

@@ -31,6 +31,9 @@ export class AfterLogoutComponent extends BasePageComponent implements OnInit {
 
     async ngOnInit() {
         await super.ngOnInit();
+        if (!this.silent) {
+            await this._authService.initialize();
+        }
         const variant = await firstValueFrom(this.variant);
         await this._authService.logoutCallback(this.silent);
         await this._router.navigate(['/' + variant.subPath]);

src/app/core/services/auth.service.ts

@@ -1,10 +1,15 @@
 import { Injectable, OnDestroy } from '@angular/core';
 import { TranslateService } from '@ngx-translate/core';
-import { ErrorResponse, OidcMetadata, User, UserManager, WebStorageStateStore } from 'oidc-client-ts';
+import { ErrorResponse, Log, OidcMetadata, User, UserManager, WebStorageStateStore } from 'oidc-client-ts';
 import { concat, firstValueFrom, map, Observable, of, Subject, Subscription } from 'rxjs';
 import { environment } from 'src/environments/environment';
 import { Variant, VariantService } from './variant.service';
 
+if (!environment.production) {
+    Log.setLogger(console);
+    Log.setLevel(Log.DEBUG);
+}
+
 @Injectable({
     providedIn: 'root',
 })
@@ -17,6 +22,7 @@ export class AuthService implements OnDestroy {
 
     private readonly _variantSubscription: Subscription;
 
+    private _initializeTriggered: boolean = false;
     private _initializedRaw: boolean = false;
     private readonly _initializedSubject = new Subject<boolean>();
 
@@ -30,11 +36,15 @@ export class AuthService implements OnDestroy {
         this._variantSubscription = this._variantService.active.subscribe((variant) =>
             this._switchUserManagerAssignment(variant),
         );
+    }
 
-        this.renew().then(() => {
+    public async initialize(): Promise<void> {
+        if (!this._initializeTriggered) {
+            this._initializeTriggered = true;
+            await this.renew();
             this._initializedRaw = true;
             this._initializedSubject.next(true);
-        });
+        }
     }
 
     private _createUserManagerAssignmentFor(variant: Variant): UserManagerAssignment {
@@ -45,6 +55,7 @@ export class AuthService implements OnDestroy {
             redirect_uri: `${uriPrefix}after-login`,
             silent_redirect_uri: `${uriPrefix}after-silent-login`,
             response_type: 'code',
+            silentRequestTimeoutInSeconds: 5000,
             scope: 'openid profile email contacts',
             userStore: new WebStorageStateStore({
                 prefix: `${variant ? variant.key : ''}.`,