Feature request: async_hooks based context

[SimenB]

Node 8 has a new async hooks module, which should be able to replace zipkin-context-cls entirely, as well as work better (i.e. at all) for promises.

It's still experimental, but not behind a flag. No polyfills exists for older nodes (lots of C++ in the implementation, so probably not incoming either).

https://nodejs.org/api/async_hooks.html

Obligatory hype blog post: https://medium.com/the-node-js-collection/async-hooks-in-node-js-illustrated-b7ce1344111f &

/cc @sveisvei (#65 (comment))

[SimenB]

Might be worth taking a look at a bridge for cls: https://www.npmjs.com/package/cls-hooked

[SimenB]

Another alternative: https://www.npmjs.com/package/asyncctx

[anujshankar]

Can use the below package instead of zipkin-context-cls for supporting async await.
https://www.npmjs.com/package/zipkin-context-cls-hooked

[Ghost---Shadow]

@anujshankar :)
#201 solves this also #135

[mziemer21]

Is there an approach that doesn't use async hooks? There currently are performance issues with async hooks.

[jcchavezs]

@mziemer21 not at the moment. Actually this is a good input for this discussion #201 (comment) cc @adriancole

[jcchavezs]

More relevant information: DataDog/dd-trace-js#695