call kill() to close socket after slim "bye" #342

Merged
merged 1 commit into from Oct 3, 2013

Projects

None yet

2 participants

Contributor
denisko commented Oct 2, 2013

Possible fix for Issue #341, though it might be to harsh. That's for you to decide.
Seems to resolve leaving sockets in CLOSE_WAIT.
Unittests and integration test run succesfully.

@amolenaar amolenaar merged commit 15ebe61 into unclebob:master Oct 3, 2013
Collaborator

Thanks, Sockets were not closed. This fix enforces everything to be terminated at the end of an test execution.

@denisko denisko deleted the denisko:connection-leak branch Oct 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment