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.

Sign up for GitHub

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

Jump to bottom

Login redirect does not honor base-path #46

Closed
tine2k opened this issue Apr 12, 2019 · 5 comments · Fixed by #47
Closed

Login redirect does not honor base-path #46

tine2k opened this issue Apr 12, 2019 · 5 comments · Fixed by #47
Labels
bug Something isn't working

Comments

@tine2k
Copy link
Contributor

tine2k commented Apr 12, 2019

I use kafkahq with security turned on and have the base-path set to "/kafka-hq".

When I browse to /kafka-hq I get redirected to /login/unauthorized instead of /kafka-hq/login/unauthorized. When using a reverse proxy the path /login/unauthorized will not serve the kafkahq login page because the prefix is missing.

Even browsing to /kafka-hq/login/unauthorized will forward /login/unauthorized.
@tchiotludo tchiotludo added the bug Something isn't working label Apr 13, 2019
@tchiotludo
Copy link
Owner

available on dev image with docker pull tchiotludo/kafkahq:dev

@tine2k
Copy link
Contributor Author

tine2k commented Apr 13, 2019

What a fast fix! Ill give it a try tonight. Thanks!

@tine2k
Copy link
Contributor Author

tine2k commented Apr 14, 2019

@tchiotludo I just tried the latest dev docker image: now it constantly redirects to /kafka-hq/login/unauthorized in a loop. Browser eventually stops with "too many redirects". I will have a closer look to why this is tonight.

@tchiotludo
Copy link
Owner

Weird, didn't see it during my test, will have a look also

@tchiotludo tchiotludo reopened this Apr 14, 2019
@tine2k
Copy link
Contributor Author

tine2k commented Apr 14, 2019

Seems to be related to my config. I am overwriting the default roles. I will further investigate.

tine2k pushed a commit to tine2k/kafkahq that referenced this issue Apr 14, 2019
fix login path to contain base path, closes tchiotludo#46
b66683f
tine2k pushed a commit to tine2k/kafkahq that referenced this issue Apr 14, 2019
fix login path to contain base path, closes tchiotludo#46
9470f6b
tine2k pushed a commit to tine2k/kafkahq that referenced this issue Apr 14, 2019
fix login path to contain base path, closes tchiotludo#46
506a136
tine2k added a commit to tine2k/kafkahq that referenced this issue Apr 14, 2019
@tine2k
fix login path to contain base path, closes tchiotludo#46
63a45cd
@tine2k tine2k mentioned this issue Apr 14, 2019
Merged
tchiotludo pushed a commit that referenced this issue Apr 14, 2019
@tine2k @tchiotludo
fix login path to contain base path, closes #46 (#47)
0704d8c
tchiotludo added a commit that referenced this issue Apr 14, 2019
@tchiotludo
fix base-path
9f279ac 
relate to #46
tchiotludo added a commit that referenced this issue Apr 15, 2019
@tchiotludo
fix base-path
71a03fb 
relate to #46
ghost pushed a commit that referenced this issue Jun 19, 2020
@goncaloValentim
Merge pull request #46 from polarising-oss/AKHQ-100-Selected-Cluster-Bug
024486d 
AKHQ-100 - Selected Cluster Bug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes login path to support configured base path tine2k/kafkahq
2 participants
@tine2k @tchiotludo