http: Upgrade/CONNECT request should detach its socket earlier
authorkoichik <koichik@improvement.jp>
Thu, 12 Jan 2012 05:16:03 +0000 (14:16 +0900)
committerkoichik <koichik@improvement.jp>
Thu, 12 Jan 2012 05:17:19 +0000 (14:17 +0900)
commit7dffbaf2ced12751b73e433b12721efa8b38ad5d
treee79cd6967292cbcc96166c8b2d0f1e2d427d7f66
parent71ae1753196f41d569d20a940ed036a50c292069
http: Upgrade/CONNECT request should detach its socket earlier

With Upgrade or CONNECT request, http.ClientRequest emits 'close' event
after its socket is closed. However, after receiving a response, the socket
is not under management by the request.

http.ClientRequest should detach the socket before 'upgrade'/'connect'
event is emitted to pass the socket to a user. After that, it should
emit 'close' event immediately without waiting for closing of the socket.

Fixes #2510.
lib/http.js
test/simple/test-http-connect.js
test/simple/test-http-upgrade-agent.js