open-telemetry/opentelemetry-specification

Clarify that API users SHOULD use valid UTF-8 for exportable data

Offen

#3.421 geöffnet am 08.09.2022

 (23 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Makefile (987 Forks)auto 404
Stalehelp wantedtriage:accepted:ready-with-sponsor

Repository-Metriken

Stars
 (4.267 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

In https://github.com/open-telemetry/opentelemetry-go/issues/3021, a user reports having spans fail to export because of invalid UTF-8.

I believe the SDK should pass this data through to the receiver, regardless of UTF-8 correctness. I am not sure whether gRPC and the Google protobuf library used by each SDK offers an option to bypass UTF-8 checking, but if so that would be one path forward if we agree.

Another option is to maintain (via sed, e.g.,) copies of the protocol that replace string with bytes; SDKs could use these definitions instead to bypass UTF-8 checking as well.

Contributor Guide