valyala/fasthttp

Request should not fail on small ReadBufferSize

已关闭

#1,873 创建于 2024年9月24日

在 GitHub 查看
 (4 条评论) (0 个反应) (0 位负责人)Go (1,755 个派生)batch import
help wantedwontfix

仓库指标

星标
 (21,741 个星标)
PR 合并指标
 (平均合并 3天) (30 天内合并 44 个 PR)

描述

Can we not fail request if the req/resp header size is greater than the read buffer size.

I can see a bufio bug was mentioned for tryRead function but looks like it's resolved now. Can we have the above mentioned behaviour?

Also, some callback could also be configured to inform that buffer size was low. Library users could emit metrics and increase the buffer size appropriately to have best balance between memory usage and CPU usage.

@valyala Could you review the request?

贡献者指南

研究方向
首先，阅读 fasthttp 的 header.go 文件，特别是第 2128 行的 tryRead 函数及其周围的缓冲区逻辑。理解 ReadBufferSize 如何影响请求解析。该问题引用了已解决的 Go bug (golang/go#14121)；研究该修复方案是否可应用于 fasthttp。此外，考虑实现一个回调机制，在缓冲区大小不足时通知用户，以便他们进行调整。检查现有注释，查看是否有维护者的反馈或额外上下文。
技术栈
gorest api
领域
backendperformance
议题类型
功能
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
Understanding of GoFamiliarity with fasthttp's bufferingReading header.go file
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
30