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

Invalid redirect after session timeout using oidc #18131

Open
tjvdmolen opened this issue May 8, 2024 · 1 comment
Open

Invalid redirect after session timeout using oidc #18131

tjvdmolen opened this issue May 8, 2024 · 1 comment
Labels
bug Something isn't working component:auth component:ui User interfaces bugs and enhancements version:2.11 Latest confirmed affected version is 2.11

Comments

@tjvdmolen
Copy link

Describe the bug
When you are using a non-dex oidc configuration and your session times out your browser will redirect you to /auth/login instead of /login and you have to clear your cookies to get out of that redirect loop.

The redirect in question seems to be:

window.location.href = `${basehref}/auth/login?return_url=${encodeURIComponent(location.href)}`;

To Reproduce

Setup argocd using a oidc provider without using dex, wait for your session to time out and trigger a call to the backend.

Expected behavior

When the session times out it should redirect to /login and the user shouldnt be stuck in a redirect loop

Version

argocd: v2.11.0+20fada8
  BuildDate: 2024-05-07T19:32:23Z
  GitCommit: 20fada836441ab22225d1253255fa11eeada876d
  GitTreeState: clean
  GoVersion: go1.22.1
  Compiler: gc
  Platform: linux/amd64
@tjvdmolen tjvdmolen added the bug Something isn't working label May 8, 2024
@rouke-broersma
Copy link
Contributor

I think this might be specific to your setup because this works fine for me. /login is the login page and /auth/login is the oidc redirect.

In this sc you can see that the /login page 'login with SSO' button redirects to /auth/login:

image

I would guess that you haven't setup your redirect url correctly in your SSO provider or something similar which creates a redirect loop.

@reggie-k reggie-k added component:auth component:ui User interfaces bugs and enhancements labels Sep 15, 2024
@andrii-korotkov-verkada andrii-korotkov-verkada added the version:2.11 Latest confirmed affected version is 2.11 label Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component:auth component:ui User interfaces bugs and enhancements version:2.11 Latest confirmed affected version is 2.11
Projects
None yet
Development

No branches or pull requests

4 participants