Skip to content

Make environment variable propagation spec compliant #4061

Description

@pellared

Is your feature request related to a problem?

Towards: open-telemetry/opentelemetry-specification#5040

The current implementation of environment variable propagation carrier is not specification compliant: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/context/env-carriers.md

At quick glance:

  • the name normalization is different
  • the env var names read from the OS are not normalized when getting the value
  • caching could be done only once

Additional context

Follows #3817

OTel .NET, Go, Java implementation should be already specification compliant.

Tip: React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.

Metadata

Metadata

Assignees

No one assigned

    Labels

    discussTo discuss in SIG meetingspec-complianceNot compliant to OpenTelemetry specstriage/acceptedIndicates an issue or PR is ready to be actively worked on.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions