Skip to content

Commit

Permalink
Clarify that lowerCamelCase field names MUST be used for OTLP/JSON (#…
Browse files Browse the repository at this point in the history
…2829)

Resolves open-telemetry/opentelemetry-specification#2795

This is a breaking change for OTLP/JSON and is allowed because OTLP/JSON is not yet Stable.
  • Loading branch information
tigrannajaryan committed Apr 27, 2023
1 parent 89968ff commit 5a5e750
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions specification/protocol/otlp.md
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,13 @@ for mapping between Protobuf and JSON, with the following deviations from that m
This aligns with the behavior of the Binary Protobuf unmarshaler and ensures that adding
new fields to OTLP messages does not break existing receivers.

- The keys of JSON objects are field names converted to lowerCamelCase. Original
field names are not valid to use a keys of JSON objects.
For example this is a valid JSON representation of a Resource:
`{ "attributes": {...}, "droppedAttributesCount": 123 }`, and this is NOT a valid
representation:
`{ "attributes": {...}, "dropped_attributes_count": 123 }`.

Note that according to [Protobuf specs](
https://developers.google.com/protocol-buffers/docs/proto3#json) 64-bit integer
numbers in JSON-encoded payloads are encoded as decimal strings, and either
Expand Down

0 comments on commit 5a5e750

Please sign in to comment.