Misc cleanup.

pkg/controller/controller.go

@@ -137,7 +137,9 @@ func (c *Controller) cachePage(ctx echo.Context, page Page, html *bytes.Buffer)
 	case err == nil:
 		log.Ctx(ctx).Debug("cached page")
 	case !context.IsCanceledError(err):
-		log.Ctx(ctx).Error("failed to cache page", "error", err)
+		log.Ctx(ctx).Error("failed to cache page",
+			"error", err,
+		)
 	}
 }
 

pkg/controller/page.go

@@ -98,8 +98,12 @@ type Page struct {
 	// This will only be populated if the request ID middleware is in effect for the given request.
 	RequestID string
 
+	// HTMX provides the ability to interact with the HTMX library
 	HTMX struct {
-		Request  htmx.Request
+		// Request contains the information provided by HTMX about the current request
+		Request htmx.Request
+
+		// Response contains values to pass back to HTMX
 		Response *htmx.Response
 	}
 

pkg/middleware/log.go

@@ -1,6 +1,7 @@
 package middleware
 
 import (
+	"fmt"
 	"strconv"
 	"time"
 
@@ -45,14 +46,6 @@ func LogRequest() echo.MiddlewareFunc {
 			sub := log.Ctx(ctx).With(
 				"ip", ctx.RealIP(),
 				"host", req.Host,
-				"method", req.Method,
-				"path", func() string {
-					p := req.URL.Path
-					if p == "" {
-						p = "/"
-					}
-					return p
-				}(),
 				"referer", req.Referer(),
 				"status", res.Status,
 				"bytes_in", func() string {
@@ -66,12 +59,18 @@ func LogRequest() echo.MiddlewareFunc {
 				"latency", stop.Sub(start).String(),
 			)
 
-			// TODO is there a (better) way to log without a message?
+			msg := fmt.Sprintf("%s %s", req.Method, func() string {
+				p := req.URL.Path
+				if p == "" {
+					p = "/"
+				}
+				return p
+			}())
 
 			if res.Status >= 500 {
-				sub.Error("")
+				sub.Error(msg)
 			} else {
-				sub.Info("")
+				sub.Info(msg)
 			}
 
 			return nil

pkg/middleware/log_test.go

@@ -3,7 +3,6 @@ package middleware
 import (
 	"context"
 	"log/slog"
-	"net/http"
 	"testing"
 
 	"github.com/labstack/echo/v4"
@@ -96,14 +95,13 @@ func TestLogRequest(t *testing.T) {
 	assert.Equal(t, "param", h.GetAttr("previous"))
 	assert.Equal(t, "21.12.12.21", h.GetAttr("ip"))
 	assert.Equal(t, "test.localhost", h.GetAttr("host"))
-	assert.Equal(t, "/abc", h.GetAttr("path"))
-	assert.Equal(t, http.MethodGet, h.GetAttr("method"))
 	assert.Equal(t, "ref.com", h.GetAttr("referer"))
 	assert.Equal(t, "200", h.GetAttr("status"))
 	assert.Equal(t, "0", h.GetAttr("bytes_in"))
 	assert.Equal(t, "5", h.GetAttr("bytes_out"))
 	assert.NotEmpty(t, h.GetAttr("latency"))
 	assert.Equal(t, "INFO", h.level)
+	assert.Equal(t, "GET /abc", h.msg)
 
 	statusCode = 500
 	exec()