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

Never allow completion handlers for async_read() and async_write() to… #128

Merged
merged 1 commit into from
Aug 11, 2016
Merged

Never allow completion handlers for async_read() and async_write() to… #128

merged 1 commit into from
Aug 11, 2016

Conversation

ignitenet-martynas
Copy link
Contributor

… run concurrently with connection_timeout_handler() because that is a race condition. Fixes #125

… run concurrently with connection_timeout_handler() because that is a race condition. Fixes #125
@ben-crowhurst
Copy link
Member

This is fantastic work! Do you have a paypal account so I can buy you a beer?

@ben-crowhurst ben-crowhurst merged commit d9aa166 into Corvusoft:master Aug 11, 2016
@ignitenet-martynas
Copy link
Contributor Author

Thanks so much for the proposition but I'm getting paid by my employer IgniteNet to do this.

Maybe some day at a proper pub :)

@ignitenet-martynas ignitenet-martynas deleted the socket_close branch August 11, 2016 20:35
@ben-crowhurst ben-crowhurst self-assigned this Apr 23, 2017
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