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

Review Socket Shutdown #16

Open
RxDave opened this issue Jun 10, 2016 · 1 comment
Open

Review Socket Shutdown #16

RxDave opened this issue Jun 10, 2016 · 1 comment
Assignees

Comments

@RxDave
Copy link
Owner

RxDave commented Jun 10, 2016

Ensure that sockets are terminated by the client rather than the server, wherever possible.
Also ensure that observable termination yields deterministic results upon graceful shutdown; e.g., the client termination notification on the server should indicate that the observable terminated, not that the client terminated or that either side was forcibly closed by the remote connection.

@RxDave RxDave self-assigned this Jun 30, 2016
@RxDave
Copy link
Owner Author

RxDave commented Jul 20, 2016

A previous release has vastly improved socket shutdown so that it's graceful and yields the appropriate reason(s). Leaving this item open for now just as a reminder to do a deeper review at some point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant