Marc Pichler

Results 446 comments of Marc Pichler

Hi @codeboten, thanks for opening this issue :slightly_smiling_face: I was going through the items on the list and checked those which we already have enabled; I left out the ones...

We're already running [CodeQL via GitHub Action](https://github.com/open-telemetry/opentelemetry-js/blob/main/.github/workflows/codeql-analysis.yml). 🙂 Vulnerability checking is something that we still need to do. We could run `npm audit --omit=dev` for that though (some devDependencies we...

I'm actually working on this as part of a major exporter overhaul that will make exporter easier to work on and test.

Not stale. A draft for this is included in #4415

It's yours @mohitk05 :slightly_smiling_face: Yes, implementation would look similar, however, you'll need to duplicate the (currently non-existent) Gauge interface in the SDK as we can't use types that are not...

> Anyone know of a workaround for now? Using an `UpDownCounter` and registering a view with a `LastValueAggregation` (see https://opentelemetry.io/docs/languages/js/instrumentation/#configure-metric-views, it does not show that example implicitly, but similar to...

As I understand it, it acts as an allow-list for attributes, filtering out all attribute keys that are not in that list. I'll open an issue on the spec to...

I added a link to the [SDK spec](https://github.com/open-telemetry/opentelemetry-specification/blob/6360b49d20ae451b28f7ba0be168ed9a799ac9e1/specification/metrics/sdk.md?plain=1#L372-L375) to the description where it mentions that it's an allow-list: > This is, at a minimum, an allow-list of attribute keys for...

@dyladan I'm currently refining issues, one question: - should we rename the package `@opentelemetry/context-async-hooks` to `@opentelemetry/context-async-local-storage` as it'll be the only context manager that's left in that package? If yes,...

Merged with the proposed changes and lint fixes in #4600