OpenTelemetry dotnet: Excessive memory allocation when parsing OpenTelemetry propagation headers

Summary

The implementation details of the baggage, B3 and Jaeger processing code in the OpenTelemetry.Api and OpenTelemetry.Extensions.Propagators NuGet packages can allocate excessive memory when parsing which could create a potential denial of service (DoS) in the consuming application.

Details

Exceeding Limits

BaggagePropagator.Inject<T>() does not enforce the length limit of 8192 characters if the injected baggage contains only one item.

This change was introduced by #1048.

Excessive allocation

The following methods eagerly allocate intermediate arrays before applying size limits.

• BaggagePropagator.Extract<T>() - this change was introduced by #1048.
• BaggagePropagator.Inject<T>() - this change was introduced by #1048.
• B3Propagator.Extract<T>() - this change was introduced by #533.
• B3Propagator.Extract<T>() - this change was introduced by #3244.
• JaegerPropagator.Extract<T>() - this change was introduced by #3309.

Impact

Excessively large propagation headers, particularly in degenerate/malformed cases that consist or large numbers of delimiter characters, can allocate excessive amounts of memory for intermediate storage of parsed content relative to the size of the original input.

Mitigation

HTTP servers often set maximum limits on the length of HTTP request headers, such as Internet Information Services (IIS) which sets a default limit of 16KB and nginx which sets a default limit of 8KB.

Workarounds

Possible workarounds include:

• Configuring appropriate HTTP request header limits.
• Disabling baggage and/or trace propagation.

Remediation

#7061 refactors the handling of baggage, B3 and Jaeger propagation headers to stop parsing eagerly when limits are exceeded and avoid allocating intermediate arrays.