-
-
Notifications
You must be signed in to change notification settings - Fork 79
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
[Android] onClose event not firing on both sockets #16
Labels
bug
Something isn't working
Comments
This comment has been minimized.
This comment has been minimized.
I have a fix ready. @micahbule, could you tell me if this is also reproducible in iOS? |
@Rapsssito I haven't tried it on iOS as I don't have a device to try it with. |
I will release an Android fix since I don't have enough time to debug iOS right now. |
Sure thing. I'll try to reproduce it on iOS if I have time. Thanks so much! |
Should be fixed in 3.1.3. Feel free to reopen this issue if the problem persists. |
Rapsssito
changed the title
Socket onClose event doesn't fire on connected device
[Android] Socket onClose event doesn't fire on connected device
Jan 10, 2020
Rapsssito
changed the title
[Android] Socket onClose event doesn't fire on connected device
[Android] onClose event not firing on both sockets
Jan 10, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
close
event for either client nor server doesn't fire when the other device disconnects.Steps to reproduce
Steps to reproduce the behavior:
close
event of the connectingsocket
.close
event of the client socket..close()
on either the socket connected to the TcpServer, or the one created by the TcpClient.Current behavior
If you call
.close()
on the socket connected to the TcpServer, only the event listener on the TcpServer will fire, not the one if the TcpClient. Vice versa.Expected behavior
Calling
.close()
on the socket will fire both theclose
event listeners registered on the server and client.Relevant information
The text was updated successfully, but these errors were encountered: