4 commits

Author	SHA1	Message	Date
Daniele Belardi	ddac3bd14f	http: add test case for req-res close ordering ... Since 55e83cbe95 has changed the ordering of the close event, add a test case. IncomingMessage will emit close before the response is sent in case the server is consuming data from it. Refs: https://github.com/nodejs/node/pull/33035#issuecomment-751489998 PR-URL: https://github.com/nodejs/node/pull/36645 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Robert Nagy <ronagy@icloud.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>	2021-01-06 11:24:14 +01:00
Robert Nagy	28e6626ce7	http: set IncomingMessage.destroyed ... IncomingMessage is a Readable stream and should properly set the destroyed property. PR-URL: https://github.com/nodejs/node/pull/33131 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>	2020-05-09 07:57:45 +02:00
Robert Nagy	8ce20aff2d	http: fix res emit close before user finish ... PR-URL: https://github.com/nodejs/node/pull/20941 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>	2018-05-24 13:23:57 -04:00
Robert Nagy	f22c7c10ca	http: always emit close on req and res ... PR-URL: https://github.com/nodejs/node/pull/20611 Fixes: https://github.com/nodejs/node/issues/20600 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>	2018-05-13 16:23:50 -07:00