Richard Simpson
Richard Simpson
This is more or less implemented in v0.4.0, but keeping this open until there's documentation.
@Schachte This SDK (currently) effectively doesn't have a concept of a "global" context and anything trying to use `@opentelemetry/api`'s concept of a context will fail. This is because until _very_...
@Schachte absolutely! Hopefully the gap between the official JS and my lib will shrink over time 😄. Happy to hear any other feedback
Yah, the Otel libs (reasonably) make a lot of assumptions that their either in a standard Node environment or a standard Web environment, both of which break down in Worker-like...
> Would it mean that the node_compat option would be required in order to use opentelemetry-sdk-workers? Or just the automatic tracing? I'm still on the fence about this and would...
> The node_compat flag is useful to run code meant for node. It would be disappointing to have to set this flag for something built for Workers. It kind of...
For sure! This is an unambiguously breaking change and requires an esoteric flag, so I want to make the reasoning is sound. I'll be updating the documentation to outline as...
Official blog post announcement: https://blog.cloudflare.com/workers-node-js-asynclocalstorage/
Admitting that workers should be very short lived and therefore it shouldn't matter that much, the only concern I have with that is it posts logs as a bundle rather...
This seems relatively easy to implement, so if there's a 👍🏻 or 👎🏻 on implementation I could contribute this.