open-telemetry/opentelemetry-specification

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

Aberta

#3.421 aberto em 8 de set. de 2022

 (23 comentários) (0 reação) (1 responsável)Makefile (987 forks)auto 404
Stalehelp wantedtriage:accepted:ready-with-sponsor

Métricas do repositório

Stars
 (4.267 estrelas)
Métricas de merge de PR
 (Mesclagem média 10d 15h) (43 fundiu PRs em 30d)

Description

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.

Guia do colaborador