Plans to merge ErrAbortHandler Fix?

[tvanderwal-lmco]

Been watching for this MR to go in for a while but haven't seen any traction on it being reviewed. Do the maintainers have intentions of getting this one in on an upcoming release?
#4336

I'm using the recovery middleware in an AI Chat backend and this panic gets printed pretty often when the client cancels a chat stream in progress. It would help clean up our logs quite a bit!

[Felipeness]

$(cat <<'ENDOFBODY'
Yeah, Gin's Recovery() is wrong here. http.ErrAbortHandler exists specifically to tell the net/http server "stop processing, tear down this connection" — it's how you kill hijacked connections, abort on client disconnects, etc. The standard library's server explicitly checks for it and skips writing a response.

But Gin's recovery middleware catches it like any other panic, writes a 500 response, and completely defeats the purpose. By the time Recovery() is done "helping," the connection that was supposed to be killed is now sitting there with a response it was never meant to get.

Workaround until this is fixed — wrap the default recovery with a check:

func CustomRecovery() gin.HandlerFunc {
    return func(c *gin.Context) {
        defer func() {
            if err := recover(); err != nil {
                // Let ErrAbortHandler do its job
                if err == http.ErrAbortHandler {
                    panic(http.ErrAbortHandler)
                }
                // Handle everything else normally
                c.AbortWithStatus(http.StatusInternalServerError)
            }
        }()
        c.Next()
    }
}

r := gin.New()
r.Use(CustomRecovery()) // instead of gin.Recovery()

This re-panics with ErrAbortHandler so it propagates up to net/http where it's actually handled correctly, while still catching real panics from your application code.

Honestly, this should be a one-line fix in Gin's recovery middleware — just check err == http.ErrAbortHandler before doing anything. If there's a PR open for this, it deserves a merge. Every Go HTTP framework that implements panic recovery needs to get this right.
ENDOFBODY
)