-
Notifications
You must be signed in to change notification settings - Fork 2
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
Bug: Keycloak redirects to frontend which redirects to keycloak ad infinitum causing "ERR_TOO_MANY_REDIRECTS", #610
Comments
It happens in my main browser (Brave), but not in incognito. Can't reproduce in Chrome. Maybe some extension thing? But still something is up. |
I also saw this on Firefox. I think something doesn't work as reliably as we thought. We'll definitely have to check this. But anyway we have to improve certain aspects of the whole login management in the browser anyway. |
Some pre-emptive logging in the middleware could help to figure out what's going on when someone manages to catch the issue. |
Hit this error after a few clicks, following a standard test flow of submission, then navigating to revision. |
I thought a little about it and tried something in https://github.com/pathoplexus/pathoplexus/pull/653. |
It seems that the preview does not work here. I'm still looking at it. |
Thanks for looking into this @JonasKellerer! Preview indeed gives me |
Is this fixed now by #638 or does this only add logging in this situation? @JonasKellerer |
Did this happen ever again or can we close this? |
I'll close this. @corneliusroemer, please reopen if this is still an issue! |
Yep no longer an issue! |
I've noticed this both in a local test and on our main deployment: after succesfully logging in as testuser on keycloak, you get redirected back and forth infinitely leading to ERR_TOO_MANY_REDIRECTS
I've tried to reproduce now on main deployment after the first time I got the bug but I can't find ways to trigger it reliably. But here are some screenshots:
Potentially reproducible if logging out then clicking on user.
The text was updated successfully, but these errors were encountered: