feat(ui): Bootstrap 5 polish — icons, D3 v7, asset cache-busting

go/app/http.go

@@ -76,6 +76,7 @@ func standardHttp(continuousDiscovery bool) {
 	router := chi.NewRouter()
 
 	// Middleware
+	router.Use(chimiddleware.StripSlashes)
 	router.Use(chimiddleware.Logger)
 	router.Use(chimiddleware.Recoverer)
 	router.Use(chimiddleware.Compress(5))
@@ -104,13 +105,23 @@ func standardHttp(continuousDiscovery bool) {
 		}
 	}
 
-	// Static file serving
+	// Static file serving. Wrap the FileServer so browsers always revalidate
+	// (via If-Modified-Since) instead of relying on heuristic freshness. The
+	// orchestrator UI ships its assets unversioned (e.g. /js/orchestrator.js),
+	// so without this header a stale JS file can persist in the browser cache
+	// long after a server upgrade.
 	prefix := config.Config.URLPrefix
 	fileServer := nethttp.FileServer(nethttp.Dir("resources/public"))
+	revalidate := func(h nethttp.Handler) nethttp.Handler {
+		return nethttp.HandlerFunc(func(w nethttp.ResponseWriter, r *nethttp.Request) {
+			w.Header().Set("Cache-Control", "no-cache, must-revalidate")
+			h.ServeHTTP(w, r)
+		})
+	}
 	if prefix != "" {
-		router.Handle(prefix+"/*", nethttp.StripPrefix(prefix, fileServer))
+		router.Handle(prefix+"/*", nethttp.StripPrefix(prefix, revalidate(fileServer)))
 	} else {
-		router.Handle("/*", fileServer)
+		router.Handle("/*", revalidate(fileServer))
 	}
 
 	if config.Config.UseMutualTLS {
@@ -171,6 +182,7 @@ func standardHttp(continuousDiscovery bool) {
 // agentsHttp starts serving agents HTTP or HTTPS API requests
 func agentsHttp() {
 	router := chi.NewRouter()
+	router.Use(chimiddleware.StripSlashes)
 	router.Use(chimiddleware.Compress(5))
 
 	if config.Config.AgentsUseMutualTLS {

go/http/render.go

@@ -19,6 +19,7 @@ package http
 import (
 	"bytes"
 	"encoding/json"
+	"fmt"
 	"html/template"
 	"net/http"
 	"os"
@@ -28,6 +29,19 @@ import (
 	"github.com/proxysql/golib/log"
 )
 
+// assetVersion is appended as ?v= on static asset URLs in templates so
+// browsers always pick up fresh JS/CSS after a server restart.
+var assetVersion = computeAssetVersion()
+
+func computeAssetVersion() string {
+	if exe, err := os.Executable(); err == nil {
+		if info, err := os.Stat(exe); err == nil {
+			return fmt.Sprintf("%d", info.ModTime().UnixNano())
+		}
+	}
+	return fmt.Sprintf("%d", os.Getpid())
+}
+
 // renderJSON writes a JSON response with the given status code.
 func renderJSON(w http.ResponseWriter, status int, data interface{}) {
 	w.Header().Set("Content-Type", "application/json; charset=UTF-8")
@@ -121,6 +135,14 @@ func renderHTML(w http.ResponseWriter, status int, name string, data interface{}
 		return
 	}
 
+	// Inject assetVersion so layout.tmpl can append ?v= on static asset URLs
+	// for cache-busting after server upgrades.
+	if m, ok := data.(map[string]interface{}); ok && m != nil {
+		if _, present := m["assetVersion"]; !present {
+			m["assetVersion"] = assetVersion
+		}
+	}
+
 	w.Header().Set("Content-Type", "text/html; charset=UTF-8")
 	w.WriteHeader(status)
 	if err := layout.Execute(w, data); err != nil {