3 commits

Author	SHA1	Message	Date
Rich Trott	fff25a0714	test: make test-http2-client-jsstream-destroy.js reliable ... Use events instead of setTimeout() calls. Fixes: https://github.com/nodejs/node/issues/36078 PR-URL: https://github.com/nodejs/node/pull/36129 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>	2020-11-17 08:28:20 -08:00
Momtchil Momtchev	adae822035	http2: centralise socket event binding in Http2Session ... Move the socket event binding to the HTTP2Session constructor so that an error event could be delivered should the constructor fail Ref: https://github.com/nodejs/node/pull/35772 PR-URL: https://github.com/nodejs/node/pull/35772 Fixes: https://github.com/nodejs/node/issues/35695 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ricky Zhou <0x19951125@gmail.com>	2020-11-08 06:28:05 -08:00
Momtchil Momtchev	6abb7e5944	http2: move events to the JSStreamSocket ... When using a JSStreamSocket, the HTTP2Session constructor will replace the socket object http2 events should be attached to the JSStreamSocket object because the http2 session handle lives there Fixes: https://github.com/nodejs/node/issues/35695 PR-URL: https://github.com/nodejs/node/pull/35772 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ricky Zhou <0x19951125@gmail.com>	2020-11-08 06:28:03 -08:00