Skip to content

Commit

Permalink
Fix security vulnerability and Dashboard's workspace detail page (#14644
Browse files Browse the repository at this point in the history
)

* activate first tab on workspace detals page if no query params were passed

Signed-off-by: Oleksii Kurinnyi <[email protected]>

* update axios version to fix high security vulnerability

Signed-off-by: Oleksii Kurinnyi <[email protected]>
  • Loading branch information
akurinnoy authored Sep 26, 2019
1 parent 8fd46cd commit ece3257
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 32 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -221,10 +221,8 @@ export class WorkspaceDetailsController {
* @param {string} tab
*/
updateSelectedTab(tab: string): void {
const value = this.tab[tab];
if (angular.isDefined(value)) {
this.selectedTabIndex = parseInt(value, 10);
}
const tabIndex = parseInt(this.tab[tab], 10);
this.selectedTabIndex = isNaN(tabIndex) ? 0 : tabIndex;
}

/**
Expand Down
40 changes: 13 additions & 27 deletions e2e/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion e2e/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
"@types/node": "^11.13.4",
"@types/rimraf": "^2.0.2",
"@types/selenium-webdriver": "^3.0.16",
"axios": "0.17.1",
"axios": "0.18.1",
"chai": "^4.2.0",
"chromedriver": "^76.0.0",
"mocha": "^6.1.4",
Expand Down

0 comments on commit ece3257

Please sign in to comment.