luxolus
/
open-vault
2
0
Fork 0
Code Issues 1 Pull Requests Releases Activity

UI: Switch usage of localStorage to sessionStorage (#14054) 

* Switch usage of localStorage to sessionStorage in UI

* Add changelog
This commit is contained in:
Chelsea Shaw 2022-02-17 10:04:53 -06:00 committed by GitHub
parent 10f19c66a7
commit a77899d276
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
changelog/14054.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:improvement
ui: Swap browser localStorage in favor of sessionStorage
```

8
ui/app/lib/local-storage.js
View File

@ -1,18 +1,18 @@
export default { export default {
getItem(key) { getItem(key) {
var item = window.localStorage.getItem(key); var item = window.sessionStorage.getItem(key);
return item && JSON.parse(item); return item && JSON.parse(item);
}, },
setItem(key, val) { setItem(key, val) {
window.localStorage.setItem(key, JSON.stringify(val)); window.sessionStorage.setItem(key, JSON.stringify(val));
}, },
removeItem(key) { removeItem(key) {
return window.localStorage.removeItem(key); return window.sessionStorage.removeItem(key);
}, },
keys() { keys() {
return Object.keys(window.localStorage); return Object.keys(window.sessionStorage);
}, },
}; };

2
ui/app/lib/token-storage.js
View File

@ -7,7 +7,7 @@ export default function (type) {
} }
let storage; let storage;
try { try {
window.localStorage.getItem('test'); localStorageWrapper.getItem('test');
storage = localStorageWrapper; storage = localStorageWrapper;
} catch (e) { } catch (e) {
storage = memoryStorage; storage = memoryStorage;

2
ui/tests/acceptance/enterprise-control-groups-test.js
View File

@ -173,7 +173,7 @@ module('Acceptance | Enterprise | control groups', function (hooks) {
await authPage.login(context.userToken); await authPage.login(context.userToken);
await settled(); await settled();
if (shouldStoreToken) { if (shouldStoreToken) {
localStorage.setItem( sessionStorage.setItem(
storageKey(accessor, 'kv/foo'), storageKey(accessor, 'kv/foo'),
JSON.stringify({ JSON.stringify({
accessor, accessor,

2
ui/tests/acceptance/redirect-to-test.js
View File

@ -47,7 +47,7 @@ module('Acceptance | redirect_to query param functionality', function (hooks) {
// normally we'd use the auth.logout helper to visit the route and reset the app, but in this case that // normally we'd use the auth.logout helper to visit the route and reset the app, but in this case that
// also routes us to the auth page, and then all of the transitions from the auth page get redirected back // also routes us to the auth page, and then all of the transitions from the auth page get redirected back
// to the auth page resulting in no redirect_to query param being set // to the auth page resulting in no redirect_to query param being set
localStorage.clear(); sessionStorage.clear();
}); });
test('redirect to a route after authentication', async function (assert) { test('redirect to a route after authentication', async function (assert) {
let url = '/vault/secrets/secret/create'; let url = '/vault/secrets/secret/create';