update readme

- repeat less
- lay framework for adding more pages

README.md

@@ -1,3 +1,17 @@
 # burning.moe
 
-Code for the burning.moe homepage
+Code for the [burning.moe](https://burning.moe) homepage
+
+## technologies used
+- [go](https://go.dev)
+  - [chi](https://github.com/go-chi/chi)
+  - [cleanenv](https://github.com/ilyakaznacheev/cleanenv)
+  - [mage](https://github.com/magefile/mage)
+
+## why?
+
+why not?
+
+Improving my go skills, and learning to write web apps in go, was a
+good opportunity to rewrite my ancient and basic website while having
+it functionally not change at all.

internal/handlers/handlers.go

@@ -2,6 +2,7 @@ package handlers
 
 import (
 	"net/http"
+	"time"
 
 	"git.burning.moe/celediel/burning.moe/internal/config"
 	"git.burning.moe/celediel/burning.moe/internal/models"
@@ -21,18 +22,12 @@ var Handlers = []Handler{
 	// /about
 	{
 		Handles: "/about",
-		Handler: func(w http.ResponseWriter, r *http.Request) {
+		Handler: makeBasicHandler("about"),
-			app.Logger.Info("Got request for about page.")
-			render.RenderTemplate(w, "about.page", &models.TemplateData{})
-		},
 	},
 	// / comes last
 	{
 		Handles: "/",
-		Handler: func(w http.ResponseWriter, r *http.Request) {
+		Handler: makeBasicHandler("home"),
-			app.Logger.Info("Got request for homepage.")
-			render.RenderTemplate(w, "home.page", &models.TemplateData{})
-		},
 	},
 }
 
@@ -40,3 +35,32 @@ var Handlers = []Handler{
 func Initialise(a *config.AppConfig) {
 	app = a
 }
+
+// makeBasicTemplateData creates a blank TemplateData containing only the
+// time the related template was generated
+func makeBasicTemplateData(name string) models.TemplateData {
+	var strMap map[string]string
+	if _, ok := app.TemplateCache.Cache[name]; ok {
+		strMap = map[string]string{
+			"GeneratedAt": app.TemplateCache.Cache[name].GeneratedAt.Format(time.UnixDate),
+		}
+	} else {
+		strMap = make(map[string]string)
+	}
+
+	templateData := models.TemplateData{
+		StringMap: strMap,
+	}
+	return templateData
+}
+
+// makeBasicHandler creates a basic handler that builds from a .page.tmpl
+// file, and sends only the time the template was generated as TemplateData
+func makeBasicHandler(name string) func(w http.ResponseWriter, r *http.Request) {
+	return func(w http.ResponseWriter, r *http.Request) {
+		app.Logger.Infof("Got request for %s page", name)
+		pageName := name + ".page.tmpl"
+		templateData := makeBasicTemplateData(pageName)
+		render.RenderTemplate(w, pageName, &templateData)
+	}
+}

internal/render/render.go

@@ -48,6 +48,7 @@ func GenerateNewTemplateCache() (models.TemplateCache, error) {
 	for _, page := range pages {
 		name := filepath.Base(page)
 		app.Logger.Info("Generating template " + name)
+		generatedAt := time.Now()
 
 		templateSet, err := template.New(name).ParseFiles(page)
 		if err != nil {
@@ -68,8 +69,9 @@ func GenerateNewTemplateCache() (models.TemplateCache, error) {
 		}
 		cache.Cache[name] = models.TemplateCacheItem{
 			Template:    templateSet,
-			GeneratedAt: time.Now(),
+			GeneratedAt: generatedAt,
 		}
+		app.Logger.Debugf("Generated %s at %v", name, generatedAt.String())
 	}
 
 	// All was good, so return the cache, and no error
@@ -77,8 +79,7 @@ func GenerateNewTemplateCache() (models.TemplateCache, error) {
 }
 
 // RenderTemplate renders requested template (t), pulling from cache.
-func RenderTemplate(w http.ResponseWriter, t string, data *models.TemplateData) {
+func RenderTemplate(w http.ResponseWriter, filename string, data *models.TemplateData) {
-	filename := t + ".tmpl"
 	var cache models.TemplateCache
 	if app.UseCache {
 		cache = app.TemplateCache
@@ -88,6 +89,7 @@ func RenderTemplate(w http.ResponseWriter, t string, data *models.TemplateData)
 		if err != nil {
 			app.Logger.Fatal("Error generating template cache, bailing out!")
 		}
+		app.TemplateCache = cache
 	}
 
 	// Get templates from cache
@@ -101,11 +103,11 @@ func RenderTemplate(w http.ResponseWriter, t string, data *models.TemplateData)
 	err := template.Template.Execute(buf, data)
 
 	if err != nil {
-		app.Logger.Fatal("Error executing template %s! Goodbye!", "err", err)
+		app.Logger.Fatal(fmt.Sprintf("Error executing template %s! Goodbye!", filename), "err", err)
 	}
 
 	_, err = buf.WriteTo(w)
 	if err != nil {
-		app.Logger.Error("Error writing template %s!\n", "err", err)
+		app.Logger.Error(fmt.Sprintf("Error writing template %s!", filename), "err", err)
 	}
 }

templates/base.layout.tmpl

@@ -20,9 +20,11 @@
 {{- block "js" . }}
 {{ end -}}
   <div id="leftfooter">
-    note: please do not set fire to cute girls
+    template generated on {{index .StringMap "GeneratedAt"}}
   </div>
   <div id="rightfooter">
+    note: please do not set fire to cute girls
+    <br />
     powered by
     <a href="https://www.debian.org/">debian</a>
     and