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.

Sign up for GitHub

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

Jump to bottom

http: make request.abort() destroy the socket #10818

Merged
merged 1 commit into from
Jan 28, 2017
Merged

http: make request.abort() destroy the socket #10818

merged 1 commit into from
Jan 28, 2017
+84 −1

Commits on Jan 28, 2017

  1. http: make request.abort() destroy the socket 

    `request.abort()` did not destroy the socket if it was called
before a socket was assigned to the request and the request
did not use an `Agent` or a Unix Domain Socket was used.

Fixes: nodejs#10812
PR-URL: nodejs#10818
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
    @lpinca
    lpinca committed Jan 28, 2017
    Configuration menu
    Copy the full SHA
    18d4ee9 View commit details
    Browse the repository at this point in the history