Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove --query.prefix option #746

Closed
yurishkuro opened this issue Mar 19, 2018 · 0 comments · Fixed by #748
Closed

Remove --query.prefix option #746

yurishkuro opened this issue Mar 19, 2018 · 0 comments · Fixed by #748

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Mar 19, 2018 •
edited
Loading

First, it is poorly named (should've been something like api-prefix), but most importantly if it is set to anything other than api it will break the UI, because it only affects the query service API routes, but does not affect how the UI JavaScript refers to them.

cc @tiffon @pavolloffay @jpkrohling
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow overriding base path for UI/API routes; rm --query.prefix jaegertracing/jaeger
1 participant
@yurishkuro