Valentin Marchaud
Valentin Marchaud
> I guess it depends what you want your fallback to look like. Would option 1 fail or return a regular int counter on platforms with no BigInt? If it...
I also think this has its place (specially since it was already existing for the k8s operator), however i would let it in the contrib repo (as auto instrumentation package)...
> They would need to be installed additionally, right Could be peer dependencies ? Also the auto instrumentation could be bundled for convenince, that was the original view of the...
I would be in favor of 2), i think it's better to have a "real" GA to avoid misunderstanding for users. (Btw i also think https://github.com/open-telemetry/opentelemetry-js/pull/2746 is preferable over https://github.com/open-telemetry/opentelemetry-js/pull/2691...
> Isn't sendBeacon and fetch both async? The only sync option is sync XHR which has problems like you mentioned I believe `sendBeacon` API is sync however the data is...
> Would an @opentelemetry/api-logs just contain the base types for now? With Noop (ConsoleLog?) Emitter. I think just having a SDK is the right way, at least thats how java...
Actually it seems that the API is on its way: https://github.com/open-telemetry/opentelemetry-specification/pull/2676
You should be able to add a configuration to ignore path starting with `/public` : ```ts new HttpInstrumentation({ ignoreIncomingRequestHook: request => { return request.url.startWith("/public") }, }) ```
`BigInt` support has landed on node in 10.4.0 (or 10.8.0 ?) but it's possible to use [jsbi](https://github.com/GoogleChromeLabs/jsbi) as a polyfill in both node and the browser if needed.
I'm okay to rename metrics since there is only one right now (and that it don't have much platform specific code) but i would prefer to keep tracing as it...