Merge 69c0ff1f8dda369485eb4009103fcceb1f5a328e into 5fad976b372e381312f8de69f0969f1284d229d3

fix: suppress http.ErrAbortHandler in recover
2026-07-12 05:51:12 +08:00 · 2025-11-16 21:13:56 +08:00 · 2025-11-16 13:13:35 +00:00
2 changed files with 6 additions and 7 deletions
--- a/recovery.go
+++ b/recovery.go
@ -68,12 +68,9 @@ func CustomRecoveryWithWriter(out io.Writer, handle RecoveryFunc) HandlerFunc {
 						}
 					}
 				}
-				if e, ok := err.(error); ok {
-					// ErrAbortHandler should be treated as broken pipe too.
-					if errors.Is(e, http.ErrAbortHandler) {
-						brokenPipe = true
-					}
-				}
+				if e, ok := err.(error); ok && errors.Is(e, http.ErrAbortHandler) {  
+					brokenPipe = true  
+				}  
 				if logger != nil {
 					const stackSkip = 3
 					if brokenPipe {
--- a/recovery_test.go
+++ b/recovery_test.go
@ -161,7 +161,9 @@ func TestPanicWithAbortHandler(t *testing.T) {
 	w := PerformRequest(router, http.MethodGet, "/recovery")
 	// TEST
 	assert.Equal(t, expectCode, w.Code)
-	assert.Contains(t, buf.String(), "net/http: abort Handler")
+	out := buf.String()
+	assert.Contains(t, out, "net/http: abort Handler")
+	assert.NotContains(t, out, "panic recovered")
 }

 func TestCustomRecoveryWithWriter(t *testing.T) {
Author	SHA1	Message	Date
Yilong Li	1d649526f8	Merge 69c0ff1f8dda369485eb4009103fcceb1f5a328e into 5fad976b372e381312f8de69f0969f1284d229d3	2025-11-16 21:13:56 +08:00
Yilong Li	69c0ff1f8d	fix: suppress http.ErrAbortHandler in recover	2025-11-16 13:13:35 +00:00