Merge branch 'agressive_redirect' into 'dev'

remove an agressive redirect to the login page. Hopefully this flows correctly

See merge request !41

src/app/launcher/launcher.component.ts

@@ -71,7 +71,7 @@ export class LauncherComponent implements OnInit {
                 ) {
       this.subscriptions = [];
       this.subscriptions.push(this.authService.sshAuthzServers.subscribe(o => {this.updateSshAuthZServers(o)}));
-      //this.subscriptions.push(this.authService.loggedInAuthZ.subscribe(o => this.navLogin(o)));
+      this.subscriptions.push(this.computeSitesService.identities.subscribe(o => this.navLogin(o)));
       this.subscriptions.push(this.computeSitesService.appidentities.subscribe(o => this.getHealth(o)));
       this.app$ = new BehaviorSubject<Strudelapp>(null);
       this.identity$ = new BehaviorSubject<Identity>(null);
@@ -193,7 +193,9 @@ export class LauncherComponent implements OnInit {
     if (ids === null) {
       if (this.computeSitesService.appidentities.value === null || this.computeSitesService.appidentities.value.length == 0) {
         this.identity$.next(null);
-        this.router.navigate(['/login']);
+        if (this.computeSitesService.appidentities.value.length == 0)  {
+          this.router.navigate(['/login']);
+        } //Don't navigate is appidentities is null. It means the observable hasn't refreshed yet, not that there are no identities
         return;
       } else {
         this.router.navigate(['/launch',this.computeSitesService.appidentities.value[0].displayName(),'accountinfo']);
@@ -202,7 +204,6 @@ export class LauncherComponent implements OnInit {
     }
     var id: Identity = this.getId(ids);
     if (id === null) {
-      console.log('id is null, callback updateIdApp');
       if (this.computeSitesService.appidentities.value === null || this.computeSitesService.appidentities.value.length == 0) {
         this.identity$.next(null);
         this.router.navigate(['/login']);