Skip to content

Make environment variable propagation spec compliant #4061

@pellared

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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions