当服务器因 gin-gonic 和 gin-contrib/gzip 出现恐慌时,HTTP 客户端收到状态码 200

HTTP client receives status code 200 when server panics with gin-gonic and gin-contrib/gzip

访问下面的gin-gonic服务器时,HTTP客户端应该收到code 500,却收到code 200。

package main

import (
    "github.com/gin-contrib/gzip"
    "github.com/gin-gonic/gin"
)

func main() {
    gin.SetMode(gin.ReleaseMode)
    r := gin.New()
    r.Use(gin.Logger())
    r.Use(gin.Recovery())
    r.Use(gzip.Gzip(gzip.DefaultCompression))

    r.POST("/test", func(c *gin.Context) {
        panic("test")                        // Server panic and client should receive code 500.
    })

    r.Run(":8080")
}

从 HTTP 客户端访问 /test 时,go 服务器日志如下所示,看起来 return 代码 500。

[GIN] 2020/09/28 - 10:23:14 | 500 |     67.2995ms |             ::1 | POST     "/test"

2020/09/28 10:23:14 [Recovery] 2020/09/28 - 10:23:14 panic recovered:
test
C:/path/to/myproject/main.go:16 (0x8f193f)
    main.func1: panic("test")

但是 HTTP 客户端收到代码 200。

当我删除 r.Use(gzip.Gzip(gzip.DefaultCompression)) 时,HTTP 客户端收到代码 500。

为什么客户端使用 r.Use(gzip.Gzip(gzip.DefaultCompression)) 收到代码 200,我该如何解决这个问题?

最后添加恢复中间件似乎可以解决这个问题。

package main

import (
    "github.com/gin-contrib/gzip"
    "github.com/gin-gonic/gin"
)

func main() {
    gin.SetMode(gin.ReleaseMode)
    r := gin.New()
    r.Use(gin.Logger())
    r.Use(gzip.Gzip(gzip.DefaultCompression))
    r.Use(gin.Recovery())

    r.POST("/test", func(c *gin.Context) {
        panic("test")                        // Server panic and client should receive code 500.
    })

    r.Run(":8080")
}

我已经重现了你的情况。邮递员收到代码 200,但服务器结果为 500。

服务器在收到post请求时会调用c.Next()执行4个处理程序。顺序如下:

gin.Logger
gin.Recovery
gzip.Gzip(gzip.DefaultCompression)
your handler

这是 gin responseWriter 写入响应 header 并且它只会写入 header 一次。

func (w *responseWriter) WriteHeaderNow() {
    if !w.Written() {
        w.size = 0
        w.ResponseWriter.WriteHeader(w.status)
    }
}

gzip.Gzip(gzip.DefaultCompression)gin.Recovery 都有延迟函数来写入响应 header。 Golang 的延迟调用按 last-in-first-out 顺序执行。因此 gzip.Gzip(gzip.DefaultCompression) 会将响应 header 写入 200,而 gin.Recovery 不会按预期将响应 header 写入 500。

所以要解决这个问题,您应该更改处理程序的顺序并确保 gin.Recovery 是最后一个要加载的处理程序。