open-telemetry / opentelemetry-cpp

The OpenTelemetry C++ Client
https://opentelemetry.io/
Apache License 2.0
811 stars 391 forks source link

[SECURITY] Remove OTLP HTTP support for TLS 1.0 and TLS 1.1, require TLS 1.2 or better #2721

Open marcalff opened 4 days ago

marcalff commented 4 days ago

Per the following RFC: https://www.ietf.org/rfc/rfc8996.html

1)

TLS 1.0 MUST NOT be used. Negotiation of TLS 1.0 from any version of TLS MUST NOT be permitted.

2)

TLS 1.1 MUST NOT be used. Negotiation of TLS 1.1 from any version of TLS MUST NOT be permitted.

In the OTLP HTTP exporter,

marcalff commented 3 days ago

Note that this issue proposes to go strait to removal of TLS 1.0 and 1.1, without announcing deprecation in opentelemetry-cpp.

Per: https://en.wikipedia.org/wiki/Transport_Layer_Security#History_and_development TLS 1.0 and TLS 1.1 have been deprecated since 2021, in https://www.ietf.org/rfc/rfc8996.html

The RFC 8996 serves as a deprecation notice, and everybody should be well aware by now.

In practice, this should not be an issue.

A system that in theory:

will be very hard to find.

Any endpoint that supports OTLP HTTP and supports SSL is expected to support TLS 1.2 already, hence the hard and abrupt removal of TLS 1.0 and TLS 1.1, for security reasons.