You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanos, Prometheus and Golang version used:
latest master (close to v0.11.0)
thanos, version 0.10.1-dev (branch: master, revision: 92c4470)
build user: pgier@pgier-laptop
build date: 20200301-00:40:46
go version: go1.13
What happened:
I set the web.route-prefix to / and this caused a redirect loop in the web interface.
Also, I tried setting a web.route-prefix without a leading / and this causes an error.
What you expected to happen:
Setting the web.route-prefix to / should have the same behaviour as the default setting of empty string.
Setting a prefix without a leading slash should either provide a cleaner input validation message or just automatically prepend a slash.
How to reproduce it (as minimally and precisely as possible):
Due to a problem in the way thanos handles a web.route-prefix set
to '/' (thanos-io/thanos#2207), the value
should only set if the user has set a value for it. If it's set
to empty string, no argument should be passed, so that thanos
uses it's own default value.
pgier
added a commit
to pgier/prometheus-operator
that referenced
this issue
Mar 2, 2020
Due to a problem in the way thanos handles a web.route-prefix set
to '/' (thanos-io/thanos#2207), the value
should only set if the user has set a value for it. If it's set
to empty string, no argument should be passed, so that thanos
uses it's own default value.
Thanos, Prometheus and Golang version used:
latest master (close to v0.11.0)
thanos, version 0.10.1-dev (branch: master, revision: 92c4470)
build user: pgier@pgier-laptop
build date: 20200301-00:40:46
go version: go1.13
What happened:
I set the web.route-prefix to
/
and this caused a redirect loop in the web interface.Also, I tried setting a web.route-prefix without a leading
/
and this causes an error.What you expected to happen:
Setting the web.route-prefix to
/
should have the same behaviour as the default setting of empty string.Setting a prefix without a leading slash should either provide a cleaner input validation message or just automatically prepend a slash.
How to reproduce it (as minimally and precisely as possible):
The text was updated successfully, but these errors were encountered: