Pass version to UI through AppConfig, instead of login payload.master

This makes the version info updated with a browser refresh (no need to logout and login again)

server/app/auth.go

@@ -63,7 +63,6 @@ func handleLogin(ds model.DataStore, username string, password string, w http.Re
 			"name":     user.Name,
 			"username": username,
 			"isAdmin":  user.IsAdmin,
-			"version":  consts.Version(),
 		})
 }
 

server/app/serve_index.go

@@ -19,22 +19,16 @@ func ServeIndex(ds model.DataStore, fs http.FileSystem) http.HandlerFunc {
 		c, err := ds.User(r.Context()).CountAll()
 		firstTime := c == 0 && err == nil
 
-		t := template.New("initial state")
-		indexHtml, err := fs.Open("index.html")
-		if err != nil {
-			log.Error(r, "Could not find `index.html` template", err)
-		}
-		indexStr, err := ioutil.ReadAll(indexHtml)
-		if err != nil {
-			log.Error(r, "Could not read from `index.html`", err)
-		}
-		t, _ = t.Parse(string(indexStr))
+		t := getIndexTemplate(r, fs)
+
 		appConfig := map[string]interface{}{
+			"version":            consts.Version(),
 			"firstTime":          firstTime,
 			"baseURL":            strings.TrimSuffix(conf.Server.BaseURL, "/"),
 			"loginBackgroundURL": conf.Server.UILoginBackgroundURL,
 		}
 		j, _ := json.Marshal(appConfig)
+
 		data := map[string]interface{}{
 			"AppConfig": string(j),
 			"Version":   consts.Version(),
@@ -45,3 +39,20 @@ func ServeIndex(ds model.DataStore, fs http.FileSystem) http.HandlerFunc {
 		}
 	}
 }
+
+func getIndexTemplate(r *http.Request, fs http.FileSystem) *template.Template {
+	t := template.New("initial state")
+	indexHtml, err := fs.Open("index.html")
+	if err != nil {
+		log.Error(r, "Could not find `index.html` template", err)
+	}
+	indexStr, err := ioutil.ReadAll(indexHtml)
+	if err != nil {
+		log.Error(r, "Could not read from `index.html`", err)
+	}
+	t, err = t.Parse(string(indexStr))
+	if err != nil {
+		log.Error(r, "Error parsing `index.html`", err)
+	}
+	return t
+}

server/app/serve_index_test.go

@@ -9,6 +9,7 @@ import (
 	"strconv"
 
 	"github.com/deluan/navidrome/conf"
+	"github.com/deluan/navidrome/consts"
 	"github.com/deluan/navidrome/model"
 	"github.com/deluan/navidrome/persistence"
 	. "github.com/onsi/ginkgo"
@@ -79,6 +80,16 @@ var _ = Describe("ServeIndex", func() {
 		config := extractAppConfig(w.Body.String())
 		Expect(config).To(HaveKeyWithValue("loginBackgroundURL", "my_background_url"))
 	})
+
+	It("sets the version", func() {
+		r := httptest.NewRequest("GET", "/index.html", nil)
+		w := httptest.NewRecorder()
+
+		ServeIndex(ds, fs)(w, r)
+
+		config := extractAppConfig(w.Body.String())
+		Expect(config).To(HaveKeyWithValue("version", consts.Version()))
+	})
 })
 
 var appConfigRegex = regexp.MustCompile(`(?m)window.__APP_CONFIG__="([^"]*)`)