Skip to content

Broadcasting logout event in order to redirect other tabs to login page #1274

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

Merged
merged 2 commits into from
May 15, 2025

Conversation

ferishili
Copy link
Contributor

This PR fixes #1194,

It introduces a broadcasting mechanism (to contain further implementation for other places as well), focusing on logout event.

On the App level we first subscribe to the broadcasting event, and on the Header => MenuUser on logout we broadcast the actual logout!

The actual tab is going to logout which redirects to j_spring_security_logout, but the other opened tabs will be redirected to login page with a 750 ms wait!

@ferishili ferishili self-assigned this May 14, 2025
@ferishili ferishili added priority:medium type:bug Something isn't working labels May 14, 2025
Copy link
Contributor

Use docker or podman to test this pull request locally.

Run test server using develop.opencast.org as backend:

podman run --rm -it -p 127.0.0.1:3000:3000 ghcr.io/opencast/opencast-admin-interface:pr-1274

Specify a different backend like stable.opencast.org:

podman run --rm -it -p 127.0.0.1:3000:3000 -e PROXY_TARGET=https://stable.opencast.org ghcr.io/opencast/opencast-admin-interface:pr-1274

It may take a few seconds for the interface to spin up.
It will then be available at http://127.0.0.1:3000.
For more options you can pass on to the proxy, take a look at the README.md.

@rlucke rlucke self-requested a review May 14, 2025 10:01
Copy link
Contributor

This pull request is deployed at test.admin-interface.opencast.org/1274/2025-05-14_10-00-49/ .
It might take a few minutes for it to become available.

Copy link
Contributor

@rlucke rlucke left a comment

Choose a reason for hiding this comment

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

looks fine to me and works as well

@Arnei Arnei merged commit 0ef62f4 into opencast:main May 15, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:medium type:bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Go to login form if user is logged out
3 participants