Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: 缓冲区长度不够, ClientHello 解析失败 #13

Merged
merged 1 commit into from
Apr 28, 2024

Conversation

neotf
Copy link
Contributor

@neotf neotf commented Apr 28, 2024

No description provided.

@XIU2
Copy link
Owner

XIU2 commented Apr 28, 2024

复现步骤是什么?给个例子我试试。

@XIU2
Copy link
Owner

XIU2 commented Apr 28, 2024

参考了一下其他项目,好像一般都是把缓冲区设置到 1500 左右。
不过提高到 2048 无非就是增加一点点内存占用,也影响不大,就当是冗余吧。

@XIU2 XIU2 merged commit 14d19f4 into XIU2:main Apr 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants