Gerhard Stöbich
Gerhard Stöbich
### Is your feature request related to a problem? Please describe. Instrumentations need to be registered before the corresponding module to instrument is loaded. Currently `NodeSDK` calls `registerInstrumentations()` as last...
followup on https://github.com/open-telemetry/opentelemetry-js/pull/161#discussion_r317842380 **Is your feature request related to a problem? Please describe.** Use of OT should not result in side effects to user application by installing event listeners which...
- [x] This only affects the JavaScript OpenTelemetry library - [ ] This may affect other libraries, but I would like to get opinions here first Followup on https://github.com/open-telemetry/opentelemetry-js-contrib/pull/549#discussion_r664733898 Currently...
### What version of OpenTelemetry are you using? 0.17.0 ### What version of Node are you using? 14.16.0 ### What did you do? see https://github.com/open-telemetry/opentelemetry-js-contrib/pull/354#discussion_r585913312 Implementing an `Instrumentation` which uses...
Maybe the information I'm looking for is already somewhere but I'm not able to find it. I'm happy if I get redirected. I'm also quite new in the ECMAModules area...
The AWS Lambda Plugin as the requirement to flush data at the time the invocation of the function is done. See https://github.com/open-telemetry/opentelemetry-js-contrib/blob/b128dae70cf5723162ef72ed0ed83924f3edf4dc/plugins/node/opentelemetry-instrumentation-aws-lambda/src/aws-lambda.ts#L199-L206 As once can see this creates a binding...
Currently tsconfig.json doesn't include a `lib` config. This results in automatically including dom - browser types (see https://www.typescriptlang.org/tsconfig#lib) The only place where this is needed is for global this [here](https://github.com/open-telemetry/opentelemetry-js-api/blob/ed9ba80427c34aa6397907eb2c28538ec74b5750/src/platform/browser/globalThis.ts#L19)...
It looks like Renovate bot is not active in this repo. At least I can't fine a single PR and we still use e.g. typescript 4.1,... Besides that I think...
Currently `NoopContextManager.active()` always returns `ROOT_CONTEXT`. This is somewhat strange if you consider following code as it looks like a basic contract is broken: ```js api.context.with(ctx, () => { api.context.active() //...
Currently tsconfig.json doesn't include a `lib` config. This results in automatically including dom - browser types (see https://www.typescriptlang.org/tsconfig#lib) The only place where this is needed is for global this [here](https://github.com/open-telemetry/opentelemetry-js-api/blob/ed9ba80427c34aa6397907eb2c28538ec74b5750/src/platform/browser/globalThis.ts#L19)...