go - 当服务器因 gin-gonic 和 gin-contrib/gzip 而崩溃时，HTTP 客户端收到状态码 200

Question

下面访问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 server 日志如下，看起来返回代码 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，我该如何解决这个问题？

Answer 1

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

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

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

这是 gin responseWriter 写响应头，它只会写一次头。

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

gzip.Gzip(gzip.DefaultCompression) 和 gin.Recovery 都有延迟函数来写入响应头。 Golang 的延迟调用是按照后进先出的顺序执行的。所以 gzip.Gzip(gzip.DefaultCompression) 会将响应 header 写入 200，而 gin.Recovery 不会按预期将响应 header 写入 500。

因此，要解决此问题，您应该更改处理程序的顺序并确保 gin.Recovery 是最后加载的处理程序。