diff --git a/app/vmalert/tpl/header.qtpl b/app/vmalert/tpl/header.qtpl index 42390fa1f9..d08ae138c8 100644 --- a/app/vmalert/tpl/header.qtpl +++ b/app/vmalert/tpl/header.qtpl @@ -1,5 +1,4 @@ {% import ( - "strings" "net/http" "path" "net/url" @@ -85,10 +84,7 @@ type NavItem struct { {% func printNavItems(r *http.Request, current string, items []NavItem) %} {%code - prefix := "/vmalert/" - if strings.HasPrefix(r.URL.Path, prefix) { - prefix = "" - } + prefix := utils.Prefix(r.URL.Path) %} <nav class="navbar navbar-expand-md navbar-dark fixed-top bg-dark"> <div class="container-fluid"> diff --git a/app/vmalert/tpl/header.qtpl.go b/app/vmalert/tpl/header.qtpl.go index 56c9002a14..0d0b46a498 100644 --- a/app/vmalert/tpl/header.qtpl.go +++ b/app/vmalert/tpl/header.qtpl.go @@ -9,52 +9,51 @@ import ( "net/http" "net/url" "path" - "strings" "github.com/VictoriaMetrics/VictoriaMetrics/app/vmalert/utils" ) -//line app/vmalert/tpl/header.qtpl:10 +//line app/vmalert/tpl/header.qtpl:9 import ( qtio422016 "io" qt422016 "github.com/valyala/quicktemplate" ) -//line app/vmalert/tpl/header.qtpl:10 +//line app/vmalert/tpl/header.qtpl:9 var ( _ = qtio422016.Copy _ = qt422016.AcquireByteBuffer ) -//line app/vmalert/tpl/header.qtpl:10 +//line app/vmalert/tpl/header.qtpl:9 func StreamHeader(qw422016 *qt422016.Writer, r *http.Request, navItems []NavItem, title string) { -//line app/vmalert/tpl/header.qtpl:10 +//line app/vmalert/tpl/header.qtpl:9 qw422016.N().S(` `) -//line app/vmalert/tpl/header.qtpl:11 +//line app/vmalert/tpl/header.qtpl:10 prefix := utils.Prefix(r.URL.Path) -//line app/vmalert/tpl/header.qtpl:11 +//line app/vmalert/tpl/header.qtpl:10 qw422016.N().S(` <!DOCTYPE html> <html lang="en"> <head> <title>vmalert`) -//line app/vmalert/tpl/header.qtpl:15 +//line app/vmalert/tpl/header.qtpl:14 if title != "" { -//line app/vmalert/tpl/header.qtpl:15 +//line app/vmalert/tpl/header.qtpl:14 qw422016.N().S(` - `) -//line app/vmalert/tpl/header.qtpl:15 +//line app/vmalert/tpl/header.qtpl:14 qw422016.E().S(title) -//line app/vmalert/tpl/header.qtpl:15 +//line app/vmalert/tpl/header.qtpl:14 } -//line app/vmalert/tpl/header.qtpl:15 +//line app/vmalert/tpl/header.qtpl:14 qw422016.N().S(`</title> <link href="`) -//line app/vmalert/tpl/header.qtpl:16 +//line app/vmalert/tpl/header.qtpl:15 qw422016.E().S(prefix) -//line app/vmalert/tpl/header.qtpl:16 +//line app/vmalert/tpl/header.qtpl:15 qw422016.N().S(`static/css/bootstrap.min.css" rel="stylesheet" /> <style> body{ @@ -114,139 +113,136 @@ func StreamHeader(qw422016 *qt422016.Writer, r *http.Request, navItems []NavItem </head> <body> `) -//line app/vmalert/tpl/header.qtpl:74 +//line app/vmalert/tpl/header.qtpl:73 streamprintNavItems(qw422016, r, title, navItems) -//line app/vmalert/tpl/header.qtpl:74 +//line app/vmalert/tpl/header.qtpl:73 qw422016.N().S(` <main class="px-2"> `) -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 } -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 func WriteHeader(qq422016 qtio422016.Writer, r *http.Request, navItems []NavItem, title string) { -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 qw422016 := qt422016.AcquireWriter(qq422016) -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 StreamHeader(qw422016, r, navItems, title) -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 qt422016.ReleaseWriter(qw422016) -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 } -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 func Header(r *http.Request, navItems []NavItem, title string) string { -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 qb422016 := qt422016.AcquireByteBuffer() -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 WriteHeader(qb422016, r, navItems, title) -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 qs422016 := string(qb422016.B) -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 qt422016.ReleaseByteBuffer(qb422016) -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 return qs422016 -//line app/vmalert/tpl/header.qtpl:76 +//line app/vmalert/tpl/header.qtpl:75 } -//line app/vmalert/tpl/header.qtpl:80 +//line app/vmalert/tpl/header.qtpl:79 type NavItem struct { Name string Url string } -//line app/vmalert/tpl/header.qtpl:86 +//line app/vmalert/tpl/header.qtpl:85 func streamprintNavItems(qw422016 *qt422016.Writer, r *http.Request, current string, items []NavItem) { -//line app/vmalert/tpl/header.qtpl:86 +//line app/vmalert/tpl/header.qtpl:85 qw422016.N().S(` `) -//line app/vmalert/tpl/header.qtpl:88 - prefix := "/vmalert/" - if strings.HasPrefix(r.URL.Path, prefix) { - prefix = "" - } +//line app/vmalert/tpl/header.qtpl:87 + prefix := utils.Prefix(r.URL.Path) -//line app/vmalert/tpl/header.qtpl:92 +//line app/vmalert/tpl/header.qtpl:88 qw422016.N().S(` <nav class="navbar navbar-expand-md navbar-dark fixed-top bg-dark"> <div class="container-fluid"> <div class="collapse navbar-collapse" id="navbarCollapse"> <ul class="navbar-nav me-auto mb-2 mb-md-0"> `) -//line app/vmalert/tpl/header.qtpl:97 +//line app/vmalert/tpl/header.qtpl:93 for _, item := range items { -//line app/vmalert/tpl/header.qtpl:97 +//line app/vmalert/tpl/header.qtpl:93 qw422016.N().S(` <li class="nav-item"> `) -//line app/vmalert/tpl/header.qtpl:100 +//line app/vmalert/tpl/header.qtpl:96 u, _ := url.Parse(item.Url) -//line app/vmalert/tpl/header.qtpl:101 +//line app/vmalert/tpl/header.qtpl:97 qw422016.N().S(` <a class="nav-link`) -//line app/vmalert/tpl/header.qtpl:102 +//line app/vmalert/tpl/header.qtpl:98 if current == item.Name { -//line app/vmalert/tpl/header.qtpl:102 +//line app/vmalert/tpl/header.qtpl:98 qw422016.N().S(` active`) -//line app/vmalert/tpl/header.qtpl:102 +//line app/vmalert/tpl/header.qtpl:98 } -//line app/vmalert/tpl/header.qtpl:102 +//line app/vmalert/tpl/header.qtpl:98 qw422016.N().S(`" href="`) -//line app/vmalert/tpl/header.qtpl:103 +//line app/vmalert/tpl/header.qtpl:99 if u.IsAbs() { -//line app/vmalert/tpl/header.qtpl:103 +//line app/vmalert/tpl/header.qtpl:99 qw422016.E().S(item.Url) -//line app/vmalert/tpl/header.qtpl:103 +//line app/vmalert/tpl/header.qtpl:99 } else { -//line app/vmalert/tpl/header.qtpl:103 +//line app/vmalert/tpl/header.qtpl:99 qw422016.E().S(path.Join(prefix, item.Url)) -//line app/vmalert/tpl/header.qtpl:103 +//line app/vmalert/tpl/header.qtpl:99 } -//line app/vmalert/tpl/header.qtpl:103 +//line app/vmalert/tpl/header.qtpl:99 qw422016.N().S(`"> `) -//line app/vmalert/tpl/header.qtpl:104 +//line app/vmalert/tpl/header.qtpl:100 qw422016.E().S(item.Name) -//line app/vmalert/tpl/header.qtpl:104 +//line app/vmalert/tpl/header.qtpl:100 qw422016.N().S(` </a> </li> `) -//line app/vmalert/tpl/header.qtpl:107 +//line app/vmalert/tpl/header.qtpl:103 } -//line app/vmalert/tpl/header.qtpl:107 +//line app/vmalert/tpl/header.qtpl:103 qw422016.N().S(` </ul> </div> </nav> `) -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 } -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 func writeprintNavItems(qq422016 qtio422016.Writer, r *http.Request, current string, items []NavItem) { -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 qw422016 := qt422016.AcquireWriter(qq422016) -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 streamprintNavItems(qw422016, r, current, items) -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 qt422016.ReleaseWriter(qw422016) -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 } -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 func printNavItems(r *http.Request, current string, items []NavItem) string { -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 qb422016 := qt422016.AcquireByteBuffer() -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 writeprintNavItems(qb422016, r, current, items) -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 qs422016 := string(qb422016.B) -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 qt422016.ReleaseByteBuffer(qb422016) -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 return qs422016 -//line app/vmalert/tpl/header.qtpl:111 +//line app/vmalert/tpl/header.qtpl:107 } diff --git a/app/vmalert/utils/links.go b/app/vmalert/utils/links.go index 870c42064e..a1e216df73 100644 --- a/app/vmalert/utils/links.go +++ b/app/vmalert/utils/links.go @@ -1,13 +1,24 @@ package utils -import "strings" +import ( + "net/url" + "strings" + + "github.com/VictoriaMetrics/VictoriaMetrics/lib/httpserver" +) const prefix = "/vmalert/" // Prefix returns "/vmalert/" prefix if it is missing in the path. func Prefix(path string) string { + pp := httpserver.GetPathPrefix() + path = strings.TrimLeft(path, pp) if strings.HasPrefix(path, prefix) { - return "" + return pp } - return prefix + res, err := url.JoinPath(pp, prefix) + if err != nil { + return path + } + return res } diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md index 98454b8411..f5dd9a427d 100644 --- a/docs/CHANGELOG.md +++ b/docs/CHANGELOG.md @@ -17,6 +17,7 @@ The following tip changes can be tested by building VictoriaMetrics components f * BUGFIX: [vmauth](https://docs.victoriametrics.com/vmauth.html): Properly handle LOCAL command for proxy protocol. See [this issue](https://github.com/VictoriaMetrics/VictoriaMetrics/issues/3335#issuecomment-1569864108). * BUGFIX: [vmalert](https://docs.victoriametrics.com/vmalert.html): Properly set datasource query params. See [this issue](https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4340). Thanks to @gsakun for [the pull request](https://github.com/VictoriaMetrics/VictoriaMetrics/pull/4341). +* BUGFIX: [vmalert](https://docs.victoriametrics.com/vmalert.html): Properly form path to static assets in WEB UI if `http.pathPrefix` set. See [this issue](https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4349). ## [v1.87.6](https://github.com/VictoriaMetrics/VictoriaMetrics/releases/tag/v1.87.6)