-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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(sqllab): don't store user in localstorage #14722
Conversation
Codecov Report
@@ Coverage Diff @@
## master #14722 +/- ##
=======================================
Coverage 77.45% 77.45%
=======================================
Files 959 959
Lines 48660 48702 +42
Branches 5734 5740 +6
=======================================
+ Hits 37688 37724 +36
- Misses 10771 10777 +6
Partials 201 201
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
/testenv up |
@pkdotson in this case I don't think a testenv will be able to fully verify the PR, see test plan |
@pkdotson Ephemeral environment spinning up at http://34.222.39.207:8080. Credentials are |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM... I really need to push for deprecating local storage and have the backend persistence always on.
Ephemeral environment shutdown and build artifacts deleted. |
* fix(sqllab): remove user info from localstorage * filter out user data when loading localstorage
* fix(sqllab): remove user info from localstorage * filter out user data when loading localstorage
* fix(sqllab): remove user info from localstorage * filter out user data when loading localstorage
* fix(sqllab): remove user info from localstorage * filter out user data when loading localstorage
SUMMARY
When the feature flag
SQLLAB_BACKEND_PERSISTENCE
is set to False, user data is currently saved in local storage. A recent change added roles information to the bootstrap user object. But with the flag off, local storage data ends up overwriting the bootstrap data, and the roles are lost, which breaks the page. This change removes user data from local storage, and prevents it from overwriting the bootstrap data.TEST PLAN
ADDITIONAL INFORMATION