reverted to main

Signed-off-by: Amol Sontakke <[email protected]>

Author: amolsontakke3576

edge/security-headers.js

@@ -43,8 +43,7 @@ function generateCSP(env, isDevServer) {
     'https://api.lfcla.dev.platform.linuxfoundation.org/',
     'https://easycla.dev.communitybridge.org/',
     'https://easycla.lfx.linuxfoundation.org/',
-    'https://contributor.easycla.lfx.linuxfoundation.org/',
-    'https://github.com/'
+    'https://contributor.easycla.lfx.linuxfoundation.org/'
   ];
   let scriptSources = [SELF, UNSAFE_EVAL, UNSAFE_INLINE,
     'https://cdn.dev.platform.linuxfoundation.org/lfx-header-v2.js',
@@ -106,8 +105,7 @@ function generateCSP(env, isDevServer) {
       'https://linuxfoundation-dev.auth0.com',
       'https://linuxfoundation-staging.auth0.com',
       'https://linuxfoundation.auth0.com',
-      'https://sso.linuxfoundation.org/',
-      'https://github.com/'
+      'https://sso.linuxfoundation.org/'
     ],
     'child-src': [],
     'media-src': [],

src/app/app-routing.module.ts

@@ -23,11 +23,6 @@ const routes: Routes = [
     pathMatch: 'full',
     component: AuthComponent
   },
-  {
-    path: 'cla/project/:projectId',
-    pathMatch: 'full',
-    component: ClaDashboardComponent
-  },
   {
     path: 'cla/project/:projectId/user/:userId',
     pathMatch: 'full',

src/app/core/models/user.ts

@@ -19,37 +19,6 @@ export class UserModel {
     version: string;
 }
 
-export class UserFromTokenModel {
-    dateCreated: string
-    dateModified: string
-    emails: any
-    lfEmail: string
-    lfUsername: string
-    userID: string
-    username: string
-    version: string
-  }
-
-  export class UserFromSessionModel {
-    date_created: string
-    date_modified: string
-    lf_email: any
-    lf_sub: any
-    lf_username: any
-    note: any
-    user_company_id: any
-    user_emails: string[]
-    user_external_id: any
-    user_github_id: string
-    user_github_username: string
-    user_gitlab_id: any
-    user_gitlab_username: any
-    user_id: string
-    user_ldap_id: any
-    user_name: string
-    version: string
-  }
-
 export class UpdateUserModel {
     companyID: string;
     dateCreated: Date;
@@ -62,4 +31,4 @@ export class UpdateUserModel {
     userID: string;
     username: string;
     version: string;
-}
+}

src/app/core/services/cla-contributor.service.ts

@@ -5,7 +5,7 @@ import {Injectable, isDevMode} from '@angular/core';
 import {HttpClient} from '@angular/common/http';
 import {Observable, Subject} from 'rxjs';
 import {Project, ProjectModel} from '../models/project';
-import {UpdateUserModel, UserFromTokenModel, UserModel} from '../models/user';
+import {UpdateUserModel, UserModel} from '../models/user';
 import {AlertService} from 'src/app/shared/services/alert.service';
 import {ActiveSignatureModel} from '../models/active-signature';
 import {IndividualRequestSignatureModel} from '../models/individual-request-signature';
@@ -70,16 +70,6 @@ export class ClaContributorService {
     return this.httpClient.get<ProjectModel>(url);
   }
 
-  getUserFromToken(): Observable<UserFromTokenModel> {
-    const url = this.getV4Endpoint('/v4/user-from-token');
-    return this.httpClient.get<UserFromTokenModel>(url);
-  }
-
-  getUserFromSession(): Observable<string> {
-    const url = this.getV2Endpoint('/v2/user-from-session');
-    return this.httpClient.get<string>(url);
-  }
-
   getUserActiveSignature(userId: string): Observable<ActiveSignatureModel> {
     const url = this.getV2Endpoint('/v2/user/' + userId + '/active-signature');
     return this.httpClient.get<ActiveSignatureModel>(url);

src/app/modules/dashboard/container/cla-dashboard/cla-dashboard.component.html

@@ -1,7 +1,7 @@
 <!-- Copyright The Linux Foundation and each contributor to CommunityBridge.
  SPDX-License-Identifier: MIT -->
 
-<app-project-title [projectId]="projectId" [userId]="userId" (successEmitter)="onAPILoad($event)" (setUserIdEmitter)="setUserId($event)"
+<app-project-title [projectId]="projectId" [userId]="userId" (successEmitter)="onAPILoad($event)"
     (errorEmitter)="hasErrorPresent($event)"></app-project-title>
 
 <div class="container-fluid dashboard-container">

src/app/modules/dashboard/container/cla-dashboard/cla-dashboard.component.ts

@@ -38,8 +38,6 @@ export class ClaDashboardComponent implements OnInit {
     this.userId = this.route.snapshot.paramMap.get('userId');
     const redirect = this.route.snapshot.queryParamMap.get('redirect');
     this.storageService.setItem(AppSettings.REDIRECT, redirect);
-    console.log('projectId'+this.projectId)
-    console.log('userId'+this.userId)
     this.hasErrorPresent();
   }
 
@@ -84,10 +82,6 @@ export class ClaDashboardComponent implements OnInit {
     }
   }
 
-  setUserId(userId: string) {
-    this.userId = userId;
-  }
-
   onAPILoad(APIType: string) {
     if (APIType === 'Project') {
       this.project = JSON.parse(this.storageService.getItem(AppSettings.PROJECT));

src/app/shared/components/project-title/project-title.component.ts

@@ -1,44 +1,33 @@
 // Copyright The Linux Foundation and each contributor to CommunityBridge.
 // SPDX-License-Identifier: MIT
 
-import { Component, Input, EventEmitter, Output, ViewChild, TemplateRef, OnInit } from '@angular/core';
+import { Component, OnInit, Input, EventEmitter, Output } from '@angular/core';
 import { AlertService } from '../../services/alert.service';
 import { ClaContributorService } from 'src/app/core/services/cla-contributor.service';
 import { ProjectModel } from 'src/app/core/models/project';
-import { UserFromSessionModel, UserFromTokenModel, UserModel } from 'src/app/core/models/user';
+import { UserModel } from 'src/app/core/models/user';
 import { StorageService } from '../../services/storage.service';
 import { AppSettings } from 'src/app/config/app-settings';
-import { NgbModal } from '@ng-bootstrap/ng-bootstrap';
-import { Router } from '@angular/router';
 
 @Component({
   selector: 'app-project-title',
   templateUrl: './project-title.component.html',
   styleUrls: ['./project-title.component.scss']
 })
 export class ProjectTitleComponent implements OnInit {
-  @ViewChild('warningModal') warningModal: TemplateRef<any>;
-
   @Input() projectId: string;
   @Input() userId: string;
   @Output() errorEmitter: EventEmitter<any> = new EventEmitter<any>();
   @Output() successEmitter: EventEmitter<any> = new EventEmitter<any>();
-  @Output() setUserIdEmitter: EventEmitter<any> = new EventEmitter<any>();
 
-  message: string;
-  redirectUrl: string;
   project = new ProjectModel();
-  userFromToken = new UserFromTokenModel();
   user = new UserModel();
-  userFromSession = new UserFromSessionModel();
+
   constructor(
     private alertService: AlertService,
     private storageService: StorageService,
     private claContributorService: ClaContributorService,
-    private router: Router
-  ) {
-
-  }
+  ) { }
 
   ngOnInit(): void {
     const hasGerrit = JSON.parse(this.storageService.getItem(AppSettings.HAS_GERRIT));
@@ -50,34 +39,47 @@ export class ProjectTitleComponent implements OnInit {
   }
 
   validateGithubFlow() {
-    console.log('validateGithubFlow'+this.projectId)
-    if (this.projectId ) {
-      this.getProject();
-      if (this.userId) {
-        // get user from user id
-        this.getUser();
+    if (this.projectId && this.userId) {
+      const localProjectId = JSON.parse(this.storageService.getItem(AppSettings.PROJECT_ID));
+      const localUserId = JSON.parse(this.storageService.getItem(AppSettings.USER_ID));
+      if (localProjectId !== this.projectId) {
+        this.getProject();
       } else {
-        // get user from token if user id is not present in url
-        this.getUserFromToken();
+        this.successEmitter.emit('Project');
+        this.project.project_name = JSON.parse(this.storageService.getItem(AppSettings.PROJECT_NAME));
+      }
+
+      if (localUserId !== this.userId) {
+        this.getUser();
       }
     } else {
       this.errorEmitter.emit(true);
-      this.alertService.error('Invalid project id in URL');
+      this.alertService.error('Invalid project id and user id in URL');
     }
   }
 
   getProject() {
-    this.claContributorService.getProject(this.projectId).subscribe(
-      (response) => {
-        this.project = response;
-        this.storageService.setItem(AppSettings.PROJECT_NAME, this.project.project_name);
-        this.storageService.setItem(AppSettings.PROJECT_ID, this.projectId);
-        this.storageService.setItem(AppSettings.PROJECT, this.project);
-        this.successEmitter.emit('Project');
-      }
-    );
+    if (this.projectId) {
+      this.claContributorService.getProject(this.projectId).subscribe(
+        (response) => {
+          this.project = response;
+          this.storageService.setItem(AppSettings.PROJECT_NAME, this.project.project_name);
+          this.storageService.setItem(AppSettings.PROJECT_ID, this.projectId);
+          this.storageService.setItem(AppSettings.PROJECT, this.project);
+          this.successEmitter.emit('Project');
+        },
+        (exception) => {
+          this.errorEmitter.emit(true);
+          this.claContributorService.handleError(exception);
+        }
+      );
+    } else {
+      this.errorEmitter.emit(true);
+      this.alertService.error('Invalid project id in URL');
+    }
   }
 
+
   getUser() {
     if (this.userId) {
       this.claContributorService.getUser(this.userId).subscribe(
@@ -97,50 +99,4 @@ export class ProjectTitleComponent implements OnInit {
       this.alertService.error('Invalid user id in URL');
     }
   }
-
-
-  getUserFromToken() {
-    this.claContributorService.getUserFromToken().subscribe(
-      (response) => {
-        console.log('getUserFromToken response ==>', response)
-        this.userFromToken = response;
-        this.storageService.setItem(AppSettings.USER_ID, this.userFromToken.userID);
-        this.storageService.setItem(AppSettings.USER, this.userFromToken);
-        this.successEmitter.emit('User');
-        this.setUserIdEmitter.emit(this.userFromToken.userID);
-        this.router.navigate(['/cla/project/' + this.projectId + '/user/' + this.userFromToken.userID]);
-      },
-      () => {
-        // If user is not found in token, get user from session
-        this.getUserFromSession();
-      }
-    );
-}
-
-  getUserFromSession() {
-    this.claContributorService.getUserFromSession().subscribe(
-      (response: any) => {
-        console.log('getUserFromSession response ==>', response)
-        this.userFromSession = response;
-        this.storageService.setItem(AppSettings.USER_ID, this.userFromSession.user_id);
-        this.storageService.setItem(AppSettings.USER, {
-          dateCreated: this.userFromSession.date_created,
-          dateModified: this.userFromSession.date_modified,
-          emails: this.userFromSession.user_emails,
-          lfEmail: this.userFromSession.lf_email,
-          lfUsername: this.userFromSession.lf_username,
-          userID: this.userFromSession.user_id,
-          username: this.userFromSession.user_name,
-          version: this.userFromSession.version
-        });
-        this.successEmitter.emit('User');
-      },
-      (exception) => {
-        console.log('getUserFromSession exception ==>', exception)
-        this.errorEmitter.emit(true);
-        this.alertService.error('An error occurred while retrieving the GitHub user from the session.');
-        this.claContributorService.handleError(exception);
-      }
-    );
-  }
-}
+}