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

Plugin route is broken with configured subdomain #6149

Closed
mariussturm opened this issue Jul 16, 2019 · 0 comments · Fixed by #6150

Comments

@mariussturm
Copy link
Member

commented Jul 16, 2019

If the web interface is configured with a subdomain directly in the server configuration all plugins have a broken route, missing the subdomain.

Expected Behavior

Plugins should automatically adjust there route to subdomain settings

Possible Solution

Steps to Reproduce (for bugs)

  1. Set a subdomain with http_publish_uri = http://127.0.0.1:9000/graylog/
  2. Try to access a part of the web interface coming from a plugin, e.g. the legacy Collector or an Enterprise plugin
  3. See the wrong route in the browser bar

Your Environment

  • Graylog Version: 3.0.2
  • Elasticsearch Version:
  • MongoDB Version:
  • Operating System:
  • Browser version:

@mariussturm mariussturm added this to the 3.0.3 milestone Jul 16, 2019

@dennisoelkers dennisoelkers self-assigned this Jul 17, 2019

dennisoelkers added a commit that referenced this issue Jul 17, 2019
Prefix plugin navigation item path with app prefix.
Due to the introduction of #5703, routes in the navigation bar are not
prefixed anymore automatically. Unfortunately this is required for
navigation items coming from plugins, as they have no knowledge about
the configured app path prefix.

This change is now automatically prefixing navigation items coming from
plugins.

Fixes #6149.
mariussturm added a commit that referenced this issue Jul 17, 2019
Prefix plugin navigation item path with app prefix. (#6150)
* Prefix plugin navigation item path with app prefix.

Due to the introduction of #5703, routes in the navigation bar are not
prefixed anymore automatically. Unfortunately this is required for
navigation items coming from plugins, as they have no knowledge about
the configured app path prefix.

This change is now automatically prefixing navigation items coming from
plugins.

Fixes #6149.

* Prefixing navigation item paths for system menu as well.
dennisoelkers added a commit that referenced this issue Jul 17, 2019
Prefix plugin navigation item path with app prefix.
Due to the introduction of #5703, routes in the navigation bar are not
prefixed anymore automatically. Unfortunately this is required for
navigation items coming from plugins, as they have no knowledge about
the configured app path prefix.

This change is now automatically prefixing navigation items coming from
plugins.

Fixes #6149.
edmundoa added a commit that referenced this issue Aug 16, 2019
Prefix plugin navigation item path with app prefix. (3.0) (#6157)
* Prefix plugin navigation item path with app prefix.

Due to the introduction of #5703, routes in the navigation bar are not
prefixed anymore automatically. Unfortunately this is required for
navigation items coming from plugins, as they have no knowledge about
the configured app path prefix.

This change is now automatically prefixing navigation items coming from
plugins.

Fixes #6149.

* Prefixing navigation item paths for system menu as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.