Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: don't allow to bypass unlock screen and access welcome screen #2928

Merged
merged 2 commits into from
Dec 18, 2023

Conversation

pavanjoshi914
Copy link
Contributor

Describe the changes you have made in this PR

security fix to not allow to bypass unlock screen and read unsensitive data

Link this PR to an issue [optional]

Fixes #2915

Type of change

(Remove other not matching type)

  • fix: Bug fix (non-breaking change which fixes an issue)

How has this been tested?

try to access welcome.html when the extension is unlocked and locked.

Checklist

  • Self-review of changed code
  • Manual testing
  • Added automated tests where applicable
  • Update Docs & Guides
  • For UI-related changes
  • Darkmode
  • Responsive layout

… welcome component

security fix to not allow to bypass unlock screen and read unsensitive data
Copy link
Contributor

@reneaaron reneaaron left a comment

Choose a reason for hiding this comment

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

tACK

@reneaaron reneaaron added the next-release To be included in the next release label Dec 18, 2023
@reneaaron reneaaron merged commit 9d5dee8 into master Dec 18, 2023
6 of 7 checks passed
@reneaaron reneaaron deleted the welcome-screen-protect branch December 18, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next-release To be included in the next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prevent opening the welcome screen when already initialized
3 participants