Refactor authentication flow

projects/common/src/fm/common-routing.module.ts

@@ -12,8 +12,7 @@ import {NotFoundComponent} from './components/not-found/not-found.component';
 const routes = [
   {
     path: 'cb',
-    component: AuthCallbackComponent,
+    component: AuthCallbackComponent
-    canActivate: [AuthCallbackGuard],
   },
   {
     path: 'loggedout',

projects/common/src/fm/components/auth-callback/auth-callback.component.ts

@@ -1,12 +1,19 @@
-import { Component } from '@angular/core';
+import { Component } from '@angular/core';
-
+import { Router } from '@angular/router';
-
+import { OAuthService } from 'angular-oauth2-oidc';
-@Component({
+import { Location} from '@angular/common';
-    selector: 'fm-auth-callback',
+
-    template:'<div></div>'
+@Component({
-})
+    selector: 'fm-auth-callback',
-export class AuthCallbackComponent {
+    template:'<div></div>'
-
+})
-  constructor() {
+export class AuthCallbackComponent {
-  }
+
-}
+  constructor(private router$: Router,private oauthService$:OAuthService) {
+    oauthService$.loadDiscoveryDocument().then(() => {
+      oauthService$.tryLoginCodeFlow().then(() => {
+        router$.navigateByUrl((oauthService$.state && oauthService$.state!="")?decodeURIComponent(oauthService$.state):"");
+      });    
+    })
+  }
+}

projects/common/src/fm/effects/app-common.effects.ts

@@ -25,7 +25,7 @@ export class AppCommonEffects {
     withLatestFrom(this.store$.select(appCommonReducers.selectGetInitialized)),
     mergeMap(([action, initialized]) => {
       var a = (action as appCommonActions.Login);
-      this.oauthService$.initImplicitFlow(a.url);
+      this.oauthService$.initCodeFlow(a.url);
       return [];
     }));
 

projects/common/src/fm/shared/app.config.factory.ts

@@ -27,17 +27,10 @@ export function appConfigFactory(injector:Injector, appConfig: AppConfig, oauthS
         oauthService.configure(authconfigFactory.getAuthConfig(appConfig));
         oauthService.setStorage(authStorage);
         oauthService.setupAutomaticSilentRefresh();
-        let location = injector.get(Location);
+      }).then(() => oauthService.loadDiscoveryDocument()
-        let router = injector.get(Router);
+      ).then(() => {
-        let urlPath = location.path();
-        oauthService.loadDiscoveryDocument().then(() => {
-          oauthService.tryLoginCodeFlow().then(() => {
-            router.navigateByUrl((oauthService.state && oauthService.state!="")?decodeURIComponent(oauthService.state):urlPath);
-          });    
-        })
-      }).then(() => {
         itemtypeService.load(appConfig).then(() => resolve()).catch(() => reject());
-      });
+      })
     });
   }
 }

src/app/app-routing.module.ts

@@ -62,7 +62,7 @@ const routes = [
 
 @NgModule({
   imports: [RouterModule.forRoot(routes,
-    { initialNavigation: false })], // ,  enableTracing: true
+    { initialNavigation: true })], // ,  enableTracing: true
   exports: [RouterModule]
 })
 export class AppRoutingModule { }