net.connect issue with Node.js 7.10.0

[watson]

A commit recently landed in Node core that was released as v7.10.0 two days ago.

It breaks async-listener and therefore also continuation-local-storage because net.connect no longer calls Socket.prototype.connect. Furthermore, it's not possible to simply monkey-patch net.connect in async-listener because it's being called internally in node by simply referencing the connect function and not the exports.connect function.

I'm working on fixing this together with @cjihrig and @jasnell. See nodejs/node#12852 for details.

It will most likely be released as v7.10.1.

Here's the relevant issue in async-listener: othiym23/async-listener#109

[watson]

The work to fix this have now been moved to nodejs/node#12861

[othiym23]

This seems to still be busted as of Node 9 -- is there something we need to do to address this in either async-listener or CLS?

[sabakugaara]

test/net-events.tap.js failed, in node8+

• platform: mac 10.12.6
• node version: 8.9.0

this may work: othiym23/async-listener#129, @watson @othiym23, I need some help, thanks!