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

Enable SSO for XDMOD and coldfront #36

Merged
merged 1 commit into from
Jul 17, 2020
Merged

Enable SSO for XDMOD and coldfront #36

merged 1 commit into from
Jul 17, 2020

Conversation

treydock
Copy link
Contributor

This uses latest ondemand-dex build that supports sessions and also latest OnDemand that allows adding new redirect URIs to default OnDemand client entry.

@aebruno
Copy link
Member

aebruno commented Jul 16, 2020

Not sure how much we care, but logging out no longer works. You can logout out of Coldfront but when you click login, you're logged back in as the same user (Due to the sticky Dex session). Same with OOD. Looks like a logout endpoint may not be supported in Dex yet. This may pose an issue for those who want to show off different roles in their application.

@dsajdak
Copy link
Contributor

dsajdak commented Jul 16, 2020

It's not ideal for the ColdFront part of the demo to not be able to login/logout multiple times as different users. However, there are work arounds if this is not possible to get working in our current setup. It may be a little confusing to the participants.

@plessbd
Copy link
Contributor

plessbd commented Jul 16, 2020

I was thinking have them use a Private browser and non Private browser.
The non private browser will be the "main" one and then the private will be for switching users.

Though we do have to think about how this will effect each others tutorials as well.

We might want to have them clear their cookies for localhost between parts.

@aebruno aebruno merged commit 99fb6fa into ubccr:master Jul 17, 2020
@aebruno
Copy link
Member

aebruno commented Jul 17, 2020

To support logging out in Coldfront we added back LDAP authentication in #42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants