github.com/gin-contrib/timeout应前置使用

首先,gin的中间件是有执行顺序的,就是按照添加的顺序进行的。之前没在意,我把timeout中间件放在了最后面,导致业务一直不正常,后面debug源码总算看明白了:

源码入口:

go 复制代码
func(c *gin.Context) {
		finish := make(chan struct{}, 1)
		panicChan := make(chan interface{}, 1)

		w := c.Writer
		buffer := bufPool.Get()
		tw := NewWriter(w, buffer)
		c.Writer = tw
		buffer.Reset()
		.....
}

然后这个NewWriter(w, buffer)的实现如下:

go 复制代码
func NewWriter(w gin.ResponseWriter, buf *bytes.Buffer) *Writer {
	return &Writer{ResponseWriter: w, body: buf, headers: make(http.Header)}
}

他这里所做的就是把原始的writer下层一级,然后new了新的body和headers,也就是进行一层封装

这里也就意味着,假如你原来已经在body和header中写入了一些内容,在后续的代码中你就无法获取到原来的内容改了,获取到的是新的header 和body

当然,gin框架自带的中间件,肯定还是考虑得很细的,所以再最后他有把新旧header和body合并的操作:

go 复制代码
			dst := tw.ResponseWriter.Header()
			for k, vv := range tw.Header() {
				dst[k] = vv
			}

			if _, err := tw.ResponseWriter.Write(buffer.Bytes()); err != nil {
				panic(err)
			}

tw.Header()就是新header,dst就是旧header,用一个for循环新header中的值合并到旧header中,也就是原始的c *gin.Context中。这里一切看起来都那么的合理是吧!

但是,这里有个bug,假设在dst中,已经包含了keyabctw.Header()也有keyabc,那么旧的key就会被覆盖掉。导致原header信息丢失,这就是我项目中遇到的bug!

同理,body也是一样存在丢失的风险。

这里他就没考虑到这个问题,所以,该中间件需要放在有业务逻辑的中间件之前执行,这样才能避免出现这个bug。

相关推荐
郑州光合科技余经理15 小时前
同城配送调度系统实战:JAVA微服务
java·开发语言·前端·后端·微服务·中间件·php
癫狂的兔子16 小时前
【BUG】【Python】【Spider】Compound class names are not allowed.
开发语言·python·bug
netkiller-BG7NYT17 小时前
阿里云语音合成CosyVoice重大BUG被我发现了
bug
321.。19 小时前
从 0 到 1 实现 Linux 下的线程安全阻塞队列:基于 RAII 与条件变量
linux·开发语言·c++·学习·中间件
马猴烧酒.2 天前
【Mybatis出现bug】应为 <statement> 或 DELIMITER,得到 ‘id‘
java·bug·mybatis
请叫我大虾2 天前
发现一个jdk中ArrayList的小BUG
android·java·bug
离离茶2 天前
【笔记1-10】Qt bug记录:dockwidget通过raise在最前面显示,toolbar的拓展菜单失效
笔记·qt·bug
测试_AI_一辰2 天前
Agent & RAG 测试工程笔记 01:Tool Calling 跑通 + 本地 PDF 接入(智谱 GLM)
人工智能·笔记·功能测试·自动化·bug
癫狂的兔子2 天前
【BUG】【Python】清除字符串空格问题
bug
癫狂的兔子4 天前
【BUG】【Python】【爬虫】爬取加载中的数据
爬虫·python·bug