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

Fix closing connection when cancelling during TLS handshake #241

Merged
merged 1 commit into from
Jul 30, 2020

Conversation

clue
Copy link
Member

@clue clue commented Jul 29, 2020

This changeset fixes closing the pending connection attempt when the connection is cancelled during the TLS handshake. This is a minor oversight from the previous cleanup in 15426bd#diff-fa535a0ca5a2f2df7b368c67e3bfd18dL66. This is now covered by some higher level integration tests to ensure correct system behavior.

@clue clue added the bug label Jul 29, 2020
@clue clue added this to the v1.6.0 milestone Jul 29, 2020
@clue clue requested review from WyriHaximus and jsor July 29, 2020 22:27
@jsor jsor merged commit 284d72d into reactphp:master Jul 30, 2020
@clue clue deleted the cancel-tls branch July 30, 2020 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants