Suppress broken-pipe and connection-reset errors on client disconnect

Author: pierres

internal/feeds/handler.go

@@ -9,6 +9,7 @@ import (
 	"archded/internal/news"
 	"archded/internal/packages"
 	"archded/internal/releases"
+	"archded/internal/ui/layout"
 )
 
 const feedItems = 25
@@ -80,7 +81,7 @@ func writeAtom(w http.ResponseWriter, feed atomFeed) {
 	_, _ = w.Write([]byte(xml.Header))
 	enc := xml.NewEncoder(w)
 	enc.Indent("", "  ")
-	if err := enc.Encode(feed); err != nil {
+	if err := enc.Encode(feed); err != nil && !layout.IsClientDisconnect(err) {
 		slog.Error("encode atom feed", "error", err)
 	}
 }

internal/feeds/planet.go

@@ -131,7 +131,7 @@ func (h *Handler) planetRSSFeed(w http.ResponseWriter, r *http.Request) {
 	_, _ = w.Write([]byte(xml.Header))
 	enc := xml.NewEncoder(w)
 	enc.Indent("", "  ")
-	if err := enc.Encode(feed); err != nil {
+	if err := enc.Encode(feed); err != nil && !layout.IsClientDisconnect(err) {
 		slog.Error("encode planet rss feed", "error", err)
 	}
 }

internal/sitemap/handler.go

@@ -88,7 +88,7 @@ func (h *Handler) index(w http.ResponseWriter, r *http.Request) {
 	if err := enc.Encode(urlSet{
 		XMLNS: "http://www.sitemaps.org/schemas/sitemap/0.9",
 		URLs:  urls,
-	}); err != nil {
+	}); err != nil && !layout.IsClientDisconnect(err) {
 		slog.Error("encode sitemap", "error", err)
 	}
 }

internal/ui/layout/render.go

@@ -5,10 +5,17 @@ import (
 	"errors"
 	"log/slog"
 	"net/http"
+	"syscall"
 
 	"github.com/a-h/templ"
 )
 
+func IsClientDisconnect(err error) bool {
+	return errors.Is(err, context.Canceled) ||
+		errors.Is(err, syscall.EPIPE) ||
+		errors.Is(err, syscall.ECONNRESET)
+}
+
 func Render(w http.ResponseWriter, r *http.Request, page Page, content templ.Component) {
 	if !page.NoIndex {
 		canonicalPath := page.CanonicalPath
@@ -24,14 +31,14 @@ func Render(w http.ResponseWriter, r *http.Request, page Page, content templ.Com
 	}
 
 	if err := Base(page, content).Render(r.Context(), w); err != nil {
-		if !errors.Is(err, context.Canceled) {
+		if !IsClientDisconnect(err) {
 			slog.Error("failed to render page", "error", err)
 		}
 	}
 }
 
 func ServerError(w http.ResponseWriter, msg string, err error) {
-	if errors.Is(err, context.Canceled) {
+	if IsClientDisconnect(err) {
 		return
 	}
 	slog.Error(msg, "error", err)