Merge pull request #186 from stapelberg/slash
Strip trailing / from push URL.
This commit is contained in:
commit
67994f1771
|
@ -336,6 +336,9 @@ func (r *registry) Push(job, instance, pushURL, method string) error {
|
||||||
if !strings.Contains(pushURL, "://") {
|
if !strings.Contains(pushURL, "://") {
|
||||||
pushURL = "http://" + pushURL
|
pushURL = "http://" + pushURL
|
||||||
}
|
}
|
||||||
|
if strings.HasSuffix(pushURL, "/") {
|
||||||
|
pushURL = pushURL[:len(pushURL)-1]
|
||||||
|
}
|
||||||
pushURL = fmt.Sprintf("%s/metrics/jobs/%s", pushURL, url.QueryEscape(job))
|
pushURL = fmt.Sprintf("%s/metrics/jobs/%s", pushURL, url.QueryEscape(job))
|
||||||
if instance != "" {
|
if instance != "" {
|
||||||
pushURL += "/instances/" + url.QueryEscape(instance)
|
pushURL += "/instances/" + url.QueryEscape(instance)
|
||||||
|
|
Loading…
Reference in New Issue