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

Some bugs are in websocket #15

Open
wants to merge 1 commit into
base: v2
Choose a base branch
from
Open

Some bugs are in websocket #15

wants to merge 1 commit into from

Conversation

anotheros
Copy link

你好,
我发现 websocket 在客户端发送数据过快的时候,服务器端会出现问题。

出现问题代码 在

应该是 github.com/gobwas/ws 在epoll 的下,读取数据导致的。

也许这个bug我该 提给 github.com/gobwas/ws 。

我看了一眼 https://github.com/gnet-io/gnet-examples/blob/v2/simple_protocol/protocol/proto.go
估计要想让 gnet完美支持 websocket ,需要自己写 codec 。

@panjf2000
Copy link
Member

能具体描述下 bug 的表现吗?

@anotheros
Copy link
Author

运行这个 #15 。即可复现。

应该是读取 websocket 帧的时候,读了一半。

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.

2 participants