szmarczak / http2-wrapper

Use HTTP/2 the same way like HTTP/1
MIT License
239 stars 18 forks source link

Investigate Node.js bugs #71

Open szmarczak opened 3 years ago

szmarczak commented 3 years ago

These issues need to be looked into before Got switches to HTTP/2 by default:

https://github.com/nodejs/node/issues/35209

Fixed

https://github.com/nodejs/node/issues/32978 https://github.com/nodejs/node/issues/35306 https://github.com/nodejs/node/issues/39400 https://github.com/nodejs/node/issues/39951 https://github.com/nodejs/node/issues/38258 https://github.com/nodejs/node/issues/35133 https://github.com/nodejs/node/issues/34532 https://github.com/nodejs/node/issues/39919 ~https://github.com/nodejs/node/issues/34311~ (https://github.com/nodejs/node/issues/34311#issuecomment-1120901208) ~https://github.com/nodejs/node/issues/29929~ (can't reproduce) ~https://github.com/nodejs/node/issues/29069~ (seems to be invalid) https://github.com/nodejs/node/issues/39363 (https://github.com/nodejs/node/pull/42340#issuecomment-1128834833) (maybe) https://github.com/nodejs/node/issues/29902 https://github.com/nodejs/node/issues/46094

szmarczak commented 5 months ago

note to myself: check if econnreset after goaway results in the pending stream has been canceled?