From 71eecb3515928675db65b79fbf970e7dfe524c55 Mon Sep 17 00:00:00 2001
From: Kyle Carberry <kyle@coder.com>
Date: Tue, 21 Mar 2023 14:04:54 -0500
Subject: [PATCH] chore: embed build info in the html to reduce requests
 (#6605)

This will reduce a request on every page load of Coder!
---
 site/index.html                               |  1 +
 site/site.go                                  | 32 ++++++++++++++-----
 .../xServices/buildInfo/buildInfoXService.ts  | 15 ++++++++-
 3 files changed, 39 insertions(+), 9 deletions(-)

diff --git a/site/index.html b/site/index.html
index 758c347fad..ac18266dcd 100644
--- a/site/index.html
+++ b/site/index.html
@@ -16,6 +16,7 @@
   <meta property="og:type" content="website" />
   <meta property="csp-nonce" content="{{ .CSP.Nonce }}" />
   <meta property="csrf-token" content="{{ .CSRF.Token }}" />
+  <meta property="build-info" content="{{ .BuildInfo }}" />
   <!-- We need to set data-react-helmet to be able to override it in the workspace page -->
   <link
     rel="alternate icon"
diff --git a/site/site.go b/site/site.go
index 194c60b466..ae3fbd5c41 100644
--- a/site/site.go
+++ b/site/site.go
@@ -6,8 +6,10 @@ import (
 	"crypto/sha1" //#nosec // Not used for cryptography.
 	_ "embed"
 	"encoding/hex"
+	"encoding/json"
 	"errors"
 	"fmt"
+	"html"
 	htmltemplate "html/template"
 	"io"
 	"io/fs"
@@ -28,6 +30,7 @@ import (
 	"golang.org/x/sync/singleflight"
 	"golang.org/x/xerrors"
 
+	"github.com/coder/coder/buildinfo"
 	"github.com/coder/coder/coderd/httpapi"
 	"github.com/coder/coder/codersdk"
 )
@@ -102,10 +105,20 @@ func Handler(siteFS fs.FS, binFS http.FileSystem, binHashes map[string]string) h
 	})))
 	mux.Handle("/", http.FileServer(http.FS(siteFS))) // All other non-html static files.
 
+	buildInfo := codersdk.BuildInfoResponse{
+		ExternalURL: buildinfo.ExternalURL(),
+		Version:     buildinfo.Version(),
+	}
+	buildInfoResponse, err := json.Marshal(buildInfo)
+	if err != nil {
+		panic("failed to marshal build info: " + err.Error())
+	}
+
 	return secureHeaders(&handler{
-		fs:        siteFS,
-		htmlFiles: files,
-		h:         mux,
+		fs:            siteFS,
+		htmlFiles:     files,
+		h:             mux,
+		buildInfoJSON: html.EscapeString(string(buildInfoResponse)),
 	})
 }
 
@@ -117,8 +130,9 @@ type handler struct {
 	// scripts, and that nonce is passed through a template.
 	// We only do this for html files to reduce the amount of in memory caching
 	// of duplicate files as `fs`.
-	htmlFiles *htmlTemplates
-	h         http.Handler
+	htmlFiles     *htmlTemplates
+	h             http.Handler
+	buildInfoJSON string
 }
 
 // filePath returns the filepath of the requested file.
@@ -138,8 +152,9 @@ func (h *handler) exists(filePath string) bool {
 }
 
 type htmlState struct {
-	CSP  cspState
-	CSRF csrfState
+	CSP       cspState
+	CSRF      csrfState
+	BuildInfo string
 }
 
 type cspState struct {
@@ -184,7 +199,8 @@ func (h *handler) ServeHTTP(resp http.ResponseWriter, req *http.Request) {
 	reqFile := filePath(req.URL.Path)
 	state := htmlState{
 		// Token is the CSRF token for the given request
-		CSRF: csrfState{Token: nosurf.Token(req)},
+		CSRF:      csrfState{Token: nosurf.Token(req)},
+		BuildInfo: h.buildInfoJSON,
 	}
 
 	// First check if it's a file we have in our templates
diff --git a/site/src/xServices/buildInfo/buildInfoXService.ts b/site/src/xServices/buildInfo/buildInfoXService.ts
index 5694886501..69d6ab0477 100644
--- a/site/src/xServices/buildInfo/buildInfoXService.ts
+++ b/site/src/xServices/buildInfo/buildInfoXService.ts
@@ -53,7 +53,20 @@ export const buildInfoMachine = createMachine(
   },
   {
     services: {
-      getBuildInfo: API.getBuildInfo,
+      getBuildInfo: async () => {
+        // Build info is injected by the Coder server into the HTML document.
+        const buildInfo = document.querySelector("meta[property=build-info]")
+        if (buildInfo) {
+          const rawContent = buildInfo.getAttribute("content")
+          try {
+            return JSON.parse(rawContent as string)
+          } catch (ex) {
+            // Ignore this and fetch as normal!
+          }
+        }
+
+        return API.getBuildInfo()
+      },
     },
     actions: {
       assignBuildInfo: assign({