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

NextCloud logout #168

Closed
mlavacca opened this issue Apr 9, 2020 · 2 comments
Closed

NextCloud logout #168

mlavacca opened this issue Apr 9, 2020 · 2 comments
Labels
kind/bug Something isn't working sig/auth sig/ui

Comments

@mlavacca
Copy link
Contributor

mlavacca commented Apr 9, 2020

This issue is probably related to #166. The logout button of NextCloud doesn't properly work. When we try to logout, there is a period of time in which the page loads something, then the NextCloud page refreshes without being logged out (the session remains open).

@alacuku
Copy link
Contributor

alacuku commented Apr 9, 2020

Nice catch! I've fixed the redirect issue: now when the logout button closes the client session with keycloak ad redirects back to crowlabs homepage.
Right now Nextcloud and Crownlabs are configured with two different clients 'k8s' for Crownlabs and 'nextcloud' for Nextcloud which means the following:

Login scenario: Assuming that we log first in Crownlabs page it will initiate an SSO session through the 'k8s' client for the user who required the session. After that all the sign in requests will not ask any more the user to enter the credentials but will use the already existin SSO session.

Logout scenario: Assuming that the user is authenticated for both Crownlabs and Nextcloud services and logsout from Crownlabs this happens only to close the session for the Crowlabs client whis is 'k8s' but all the existing sessions of the other clients will not be affected by this action. This is the reason why a service works fine even if we logout from the other one.

Now a logout in Nextcloud or Crownlabs closes all the sessions for the user which means the user has to authenticate again.

@alacuku
Copy link
Contributor

alacuku commented Apr 14, 2020

Fixed the logout redirect. Closing the issue.

@alacuku alacuku closed this as completed Apr 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working sig/auth sig/ui
Projects
None yet
Development

No branches or pull requests

3 participants