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

ArgoCD UI should redirect to the /login page when user not logged in #20874

Open
3 tasks done
narcistesa opened this issue Nov 20, 2024 · 2 comments · May be fixed by #20878
Open
3 tasks done

ArgoCD UI should redirect to the /login page when user not logged in #20874

narcistesa opened this issue Nov 20, 2024 · 2 comments · May be fixed by #20878
Labels
bug Something isn't working more-information-needed Further information is requested version:2.13 Latest confirmed affected version is 2.13

Comments

@narcistesa
Copy link

Checklist:

  • I've searched in the docs and FAQ for my answer: https://bit.ly/argocd-faq.
  • I've included steps to reproduce the bug.
  • I've pasted the output of argocd version.

Describe the bug

The ArgoCD UI replaces the browser history to navigate to /login?return_url=... without reloading the page. This behaviour is causing issues for us since we want to rewrite the /login location to /auth/login in our ingress controller. It would help immensely if the history.push was replaced by a window.location.href here: https://github.com/argoproj/argo-cd/blob/master/ui/src/app/app.tsx#L270

Not sure if this should be a feature request or a bug.

To Reproduce

Navigate to the / location of the ArgoCD UI and notice that the browser location has been updated to /login?return_url=... without a page reload. Checking the ingress controller logs shows that no calls were made to argocd-server.

Expected behavior

Ideally this should trigger a page reload to /login?return_url=... instead of just manipulating the browser history.

Screenshots

N/A

Version

~ ❯ argocd version
argocd: v2.13.1+af54ef8
  BuildDate: 2024-11-20T18:32:21Z
  GitCommit: af54ef8db5adfa77a08d4d05b1318a2198084c22
  GitTreeState: clean
  GoVersion: go1.23.3
  Compiler: gc
  Platform: darwin/arm64
argocd-server: v2.12.6+4dab5bd```

**Logs**

N/A

@narcistesa narcistesa added the bug Something isn't working label Nov 20, 2024
@andrii-korotkov-verkada andrii-korotkov-verkada added the version:2.13 Latest confirmed affected version is 2.13 label Nov 21, 2024
@andrii-korotkov-verkada
Copy link
Contributor

What's the return_url value?

@andrii-korotkov-verkada andrii-korotkov-verkada added the more-information-needed Further information is requested label Nov 22, 2024
@narcistesa
Copy link
Author

What's the return_url value?

See the PR please, it's the same as before, I just replaced history.push with window.location.href: https://github.com/argoproj/argo-cd/pull/20878/files#diff-b15a692b8ae47d49f16da31065911218896c6fba99f2d4449f94060de0590f84R270

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working more-information-needed Further information is requested version:2.13 Latest confirmed affected version is 2.13
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants