stream: `autoDestroy: true`

[ronag]

This is a tracking issue for migrating all node core modules to use autoDestroy: true

• _http_incoming.js
• internal/http2/core.js
• internal/http2/compat.js
• net.js net: autoDestroy Socket #31806
• internal/fs/streams.js (stream: construct #29656)
• zlib.js

[ronag]

I think to a certain degree this would be simpler if done at the same time as migrating to construct (if that gets merged).

[ronag]

Related #30621, #30623

[ronag]

Crossed out http2. Does not seem feasible to move to http2. Unsure if http2 lifecycle adheres to the default streams life cycle.

[dnlup]

Is _http_ougoing.js needed? I think _http_incoming.js can be marked as done (#30623)

[ronag]

Is _http_ougoing.js needed?

Is not a real stream, yet.

I think _http_incoming.js can be marked as done (#30623)

No, it's still not using autoDestroy, https://github.com/nodejs/node/pull/30623/files#diff-f7d8a311cc7ac30522e414465bda2229R51

[dnlup]

Thank you @ronag for the clarification. I misinterpreted this issue by considering that the modules should have been just refactored to be aware of the new default autoDestroy: true option in streams. I would like to help with _http_incoming.js then, I'll try to work on it.