Coverity-CWE 922: Insecure Storage of Sensitive Information(localStorage write)
- Coverity fix for localStorage write issue: For storing datas
in browser localStorage so changed it to sessionStorage
Change-Id: I5179dbd864d9cf6c9f82fdb54f4ad2544e945ac0
Signed-off-by: SANDHYA.JS <sandhya.j@tataelxsi.co.in>
diff --git a/src/services/ProjectService.ts b/src/services/ProjectService.ts
index c22f33a..0473e6b 100644
--- a/src/services/ProjectService.ts
+++ b/src/services/ProjectService.ts
@@ -77,13 +77,13 @@
/** Get current project details from local storage @public */
public getCurrentProjectDetails(): Observable<{}> {
- const project: string = localStorage.getItem('project_id');
+ const project: string = sessionStorage.getItem('project_id');
return this.restService.getResource(environment.PROJECTS_URL + '/' + project);
}
/** Returns all the projects for a particular users @public */
public getUserProjects(): Observable<{}> {
- const username: string = localStorage.getItem('username');
+ const username: string = sessionStorage.getItem('username');
return this.restService.getResource(environment.USERS_URL + '/' + username);
}
@@ -92,8 +92,8 @@
this.getUserProjects().subscribe((projects: UserDetail) => {
const projectList: {}[] = projects.project_role_mappings;
projectList.filter((list: ProjectModel) => {
- if (list.project === localStorage.getItem('project_id')) {
- localStorage.setItem('project', list.project_name);
+ if (list.project === sessionStorage.getItem('project_id')) {
+ sessionStorage.setItem('project', list.project_name);
this.authService.projectName$.next(list.project_name);
}
});
@@ -107,7 +107,7 @@
/** Toggle projects on selection @public */
public switchProjectModal(list: ProjectData): void {
- const username: string = localStorage.getItem('username');
+ const username: string = sessionStorage.getItem('username');
this.modalService.open(SwitchProjectComponent, { backdrop: 'static' })
.componentInstance.params = { projectID: list.project, username };
}