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

Bad redirect if rest and web hosted on same addr #2593

Closed
hc4 opened this issue Aug 3, 2016 · 5 comments
Closed

Bad redirect if rest and web hosted on same addr #2593

hc4 opened this issue Aug 3, 2016 · 5 comments
Labels
Milestone

Comments

@hc4
Copy link
Contributor

@hc4 hc4 commented Aug 3, 2016

Problem description

I have rest_listen_uri = http://0.0.0.0:80/ and web_listen_uri = http://0.0.0.0:80/web
If user opens http://graylog, he will be redirected to http://0.0.0.0:80/web

Environment

  • Graylog Version: 2.1.0-beta2
@joschi joschi added the bug label Aug 3, 2016
@shishkin
Copy link

@shishkin shishkin commented Aug 3, 2016

web_transport_uri property seem gone and has no effect.

@joschi
Copy link
Contributor

@joschi joschi commented Aug 3, 2016

@shishkin There never was such a configuration setting.

@shishkin
Copy link

@shishkin shishkin commented Aug 3, 2016

I see, there was no redirect from REST to UI as well. Now when REST redirects to UI it doesn't know what the external URI should be. Maybe it would be more straightforward to have a common setting for external host (http_external_host?) instead of rest_transport_uri that would influence both REST and UI.

@shishkin
Copy link

@shishkin shishkin commented Aug 3, 2016

@hc4 web_endpoint_uri (set as GRAYLOG_WEB_ENDPOINT_URI docker env var) doesn't make a difference, curl -i -H'Accept: text/html' http://localhost:12900 still redirects to http://0.0.0.0:12900/web, which is the value of GRAYLOG_WEB_LISTEN_URI. Same happens if I set GRAYLOG_WEB_ENDPOINT_URI to http://127.0.0.1:12900/web

@shishkin
Copy link

@shishkin shishkin commented Aug 3, 2016

Although the redirect is broken, the UI works fine with @hc4 's suggestion (#2595 (comment)) when opening http://localhost:12900/web/ (trailing / is important, without it will redirect to the listen IP).

@kroepke kroepke added this to the 2.1.0 milestone Aug 8, 2016
@kroepke kroepke added S4 P3 labels Aug 8, 2016
@kroepke kroepke modified the milestone: 2.1.0 Aug 8, 2016
@joschi joschi closed this in #2675 Aug 16, 2016
@kroepke kroepke added the triaged label Sep 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

4 participants
You can’t perform that action at this time.