opentelemetry-js-api
opentelemetry-js-api copied to clipboard
open-telemetry
OpenTelemetry Javascript API
That typescript option only causes an issue if your import statement doesn't match the case of the file you're importing. It doesn't make different files have consistent casing. I agree...
This is probably not a big issue. The current `setGlobalPropagator` and `getGlobalPropagator` only accept the `TextMapPropagator` type but it is possible that there can be other propagator types such as...
These two are default propagators and currently live `@opentelemetry/core`. It is not clear what is part of `core` since it also has sampling implementations. It would be clear if they...
Problem 1: just setting the logger uses a default level instead of the one from env configuration (). Which means when setting a logger you need to fetch the ENV...
The upgrade guidelines have sections for upgrading: * from version 0.21.0 to 1.0.0 - makes sense * from version 0.20.0 to 0.21.0 - also makes sense * from version 1.0.0-rc.3...
It would be useful to get an AMD/Require source option available in NPM form to avoid a transpilation step for developers. If we provide a dev framework that supports AMD...
Create a troubleshooting section on the readme which is a little more prominent
https://github.com/open-telemetry/opentelemetry-js-api/blob/main/src/internal/global-utils.ts uses diag logger to report issues with registering globals. This is not adequate when the user doesn't control both of the installed packages registering those globals(that is definitely most...
`Span#context` was renamed in https://github.com/open-telemetry/opentelemetry-js-api/pull/45. For consistency, it would be nice if link did the same. https://github.com/open-telemetry/opentelemetry-js-api/blob/9b3f9c75a57190841b5f091933932386f63a62ac/src/trace/link.ts#L37 This is a minor ask – if we don't want to create more...
Diag logger has a really crucial part while debugging a host - for tracing issues or even a non-related logic issue. Currently, the logs dumped by the diag logger have...
