fix: Fix 500 errors when clients disconnect

When clients disconnect, we now logging 500 error due to
Context.Cancelled error.

This leads to alerts and uses up our error budgets.

This reverts merge request !676

Changelog: fixed

1 files changed, 2 insertions, 9 deletions

diff --git a/internal/vfs/serving/serving.go b/internal/vfs/serving/serving.go
index 57b0228f..6f45e1d5 100644
--- a/internal/vfs/serving/serving.go
+++ b/internal/vfs/serving/serving.go
@@ -47,17 +47,10 @@ func serveContent(w http.ResponseWriter, r *http.Request, modtime time.Time, con
 		return
 	}
 
-	if r.Method == http.MethodHead {
-		w.WriteHeader(code)
-		return
-	}
+	w.WriteHeader(code)
 
 	if r.Method != http.MethodHead {
-		if _, err := io.Copy(w, content); err != nil {
-			errortracking.Capture(err, errortracking.WithRequest(r), errortracking.WithStackTrace())
-			logging.LogRequest(r).WithError(err).Error("error serving content")
-			httperrors.Serve500(w)
-		}
+		io.Copy(w, content)
 	}
 }