open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
2.9k stars 2.27k forks source link

Explicitly define processors that drop the client context #34453

Open dmitryax opened 1 month ago

dmitryax commented 1 month ago

We have a few processors that drop client metadata (context), which potentially can carry important authentication data that exporters are supposed to use.

It's important to clearly document the processors dropping the context. Can be done in metadata.yaml as another warning added to https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/standard-warnings.md#unsound-transformations.

Action items

  1. Identify processors dropping the context
  2. Fix those that drop it unintentionally
  3. Document those that drop it intentionally
Neema-Joju commented 1 month ago

Hi, May I work on this issue? It would be my first time contributing to this repo. Thanks