From 59cf185143c9e36ae0e9d8fd27f7ef962b1188bb Mon Sep 17 00:00:00 2001 From: Catherine Date: Thu, 23 Apr 2026 16:42:46 +0000 Subject: [PATCH] Only log media type for `PUT`, `PATCH`, and `POST` requests. There isn't much point in logging `Accept:` for GET requests and it is very noisy. --- src/pages.go | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/pages.go b/src/pages.go index b5324d8..b5dc18d 100644 --- a/src/pages.go +++ b/src/pages.go @@ -846,14 +846,13 @@ func ServePages(w http.ResponseWriter, r *http.Request) { if config.Audit.IncludeIPs != "" { GetPrincipal(r.Context()).IpAddress = proto.String(r.RemoteAddr) } - var mediaType string switch r.Method { - case "HEAD", "GET": - mediaType = r.Header.Get("Accept") + case "PUT", "PATCH", "POST": + mediaType := r.Header.Get("Content-Type") + logc.Println(r.Context(), "pages:", r.Method, r.Host, r.URL, mediaType) default: - mediaType = r.Header.Get("Content-Type") + logc.Println(r.Context(), "pages:", r.Method, r.Host, r.URL) } - logc.Println(r.Context(), "pages:", r.Method, r.Host, r.URL, mediaType) if hostname, err := os.Hostname(); err == nil { if region := os.Getenv("PAGES_REGION"); region != "" { w.Header().Add("Server", fmt.Sprintf("git-pages (%s; %s)", region, hostname))