Skip to content

Clear the sessionStorage when logging out#1003

Merged
peternied merged 1 commit intoopensearch-project:mainfrom
AMoo-Miki:clear-sessionstorage
Jun 1, 2022
Merged

Clear the sessionStorage when logging out#1003
peternied merged 1 commit intoopensearch-project:mainfrom
AMoo-Miki:clear-sessionstorage

Conversation

@AMoo-Miki
Copy link
Copy Markdown
Contributor

Signed-off-by: Miki miki@amazon.com

Description

OSD uses the browser's sessionStorage to store potentially sensitive information. This change clears the sessionStorage during logout.

Category

Enhancement

Why these changes are required?

Without this, potentially sensitive information is left in the sessionStorage.

Check List

  • New functionality includes testing
  • New functionality has been documented
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Miki <miki@amazon.com>
@AMoo-Miki AMoo-Miki requested a review from a team May 31, 2022 23:56
Copy link
Copy Markdown

@seanneumann seanneumann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@peternied peternied left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making this change @AMoo-Miki

@peternied peternied merged commit 0d878d6 into opensearch-project:main Jun 1, 2022
spartan2015 pushed a commit to spartan2015/security-dashboards-plugin that referenced this pull request Aug 8, 2022
Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: Vasile Negru <vasile@eosfintek.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants