Skip to content
This repository was archived by the owner on Jul 14, 2026. It is now read-only.

switched to sessionStorage - #2416

Merged
DeepDiver1975 merged 3 commits into
owncloud:masterfrom
mrkwenzel:feature/oics-session-storage
Nov 14, 2019
Merged

switched to sessionStorage#2416
DeepDiver1975 merged 3 commits into
owncloud:masterfrom
mrkwenzel:feature/oics-session-storage

Conversation

@mrkwenzel

Copy link
Copy Markdown
Contributor

Description

Switched the storage of the auth service from local to session storage.

Related Issue

Motivation and Context

If the browser will be closed and therefore the session ends the user should start a new auth flow.

Open tasks:

  • Check if there are some more places where the tokens are stored in the local storage

@LukasHirt

Copy link
Copy Markdown
Collaborator

We also store user + capabilities in the local storage via vuex store. I think that will need to be adjusted as well.

see https://github.com/owncloud/phoenix/blob/8cc4dbae38e159be89e1586c3ab4ab2f4c4e874b/src/store/index.js#L15

@LukasHirt

Copy link
Copy Markdown
Collaborator

IMO looks good. Still would prefer to see @DeepDiver1975 opinion here though before approving.

@ownclouders

Copy link
Copy Markdown
Contributor
💥 Acceptance tests Trashbin failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6317/

20191111-110354-875.png
20191111-110355-172.png
20191111-110355-431.png
20191111-110355-791.png
20191111-110356-136.png
20191111-111112-571.png
20191111-111245-069.png
20191111-111456-473.png
20191111-111456-967.png
20191111-111457-310.png
20191111-111457-754.png
20191111-111458-007.png
20191111-111633-812.png
20191111-111634-120.png
20191111-111634-438.png
20191111-111634-691.png
20191111-111635-020.png
20191111-111635-445.png
20191111-111636-023.png
20191111-111636-347.png
20191111-111636-724.png
20191111-111637-026.png
20191111-111637-279.png
20191111-111637-624.png
20191111-111637-956.png
20191111-111638-347.png
20191111-111638-635.png
20191111-112006-921.png
20191111-112007-181.png
20191111-112007-505.png
20191111-112007-770.png
20191111-112007-994.png

@DeepDiver1975
DeepDiver1975 merged commit d13500f into owncloud:master Nov 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move tokens from local storage to session storage

4 participants