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

Frontend should detect when the server has logged the user out #1690

Closed
Tracked by #1543
mrnicegyu11 opened this issue Aug 27, 2024 · 2 comments · Fixed by ITISFoundation/osparc-simcore#6261
Closed
Tracked by #1543
Assignees
Labels
a:frontend type:enhancement Feature requests (functionality and UI)
Milestone

Comments

@mrnicegyu11
Copy link
Member

mrnicegyu11 commented Aug 27, 2024

On 26Aug2024, Taylor's computer froze and he turned it on again only a longer time (1h?) after this. His browser detected the crash and Taylor restored all tabs, including osparc. He apparently managed to get back to the dashboard of his open project, but the osparc server appareantly has already logged his user session out. Subsequent requests to save his project (the postgres DB entry, POST request) were responded with Unauthorized.
I would have been an improved experience for TN if he had gotten a "red cloud" or some notification that you are logged out.

CC @newton1985

@mrnicegyu11
Copy link
Member Author

I have compiled what happened into an issue and assign all frontend devs, this should be adressed with priority as part of the special PHRT support case for this sprint.

@odeimaiz
Copy link
Member

odeimaiz commented Aug 28, 2024

We agreed that whenever the frontend receives a 401 Unauthorized, it will make a call to the /me endpoint and if it receives yet another 401, it will assume it was logged out in the backend, and therefore it will log the user out from the application.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:frontend type:enhancement Feature requests (functionality and UI)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants
@jsaq007 @ignapas @mrnicegyu11 @odeimaiz and others