Skip to content
This repository has been archived by the owner on Sep 25, 2023. It is now read-only.

hybridconnector设置心跳遇到的问题 #814

Closed
yc90s opened this issue Mar 24, 2016 · 2 comments · Fixed by #815
Closed

hybridconnector设置心跳遇到的问题 #814

yc90s opened this issue Mar 24, 2016 · 2 comments · Fixed by #815

Comments

@yc90s
Copy link
Contributor

yc90s commented Mar 24, 2016

我给connector的heartbeat设置为1，当客户端连接上来后，我将客户端的网络禁止掉，查看log，发现服务器一直不会主动关闭这个session，超时机制没生效？
一直要到客户端重新恢复网络后，服务器才会将这个session关闭
@yc90s
Copy link
Contributor Author

yc90s commented Mar 24, 2016

我看源码发现了问题在哪
pomelo的心跳是由客户端发起的，服务端被动响应，这种做法没有什么作用啊
客户端要是不给服务器发心跳包（像上面的直接断网），服务器也没去清理这个session

@yc90s yc90s mentioned this issue Mar 24, 2016
Merged
fantasyni added a commit that referenced this issue Jan 20, 2017
@fantasyni
Merge pull request #815 from 1050676515/master
90a16ba 
#814 fix heartbeat bug
@fantasyni
Copy link
Member

合并了，多谢

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#814 fix heartbeat bug yc90s/pomelo
2 participants
@fantasyni @yc90s