Skip to content

Commit

Permalink
Fix the unclear SHOULD requirement for "gzip" OTLP content type (#1038)
Browse files Browse the repository at this point in the history
MUST is the correct requirement here. There is no situation when a different content type is a better choice.
  • Loading branch information
tigrannajaryan authored and VinozzZ committed Jun 21, 2024
1 parent 25eb61c commit 90245b8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion specification/protocol/otlp.md
Original file line number Diff line number Diff line change
Expand Up @@ -323,7 +323,7 @@ The client MUST set "Content-Type: application/x-protobuf" request header when
sending binary-encoded Protobuf or "Content-Type: application/json" request
header when sending JSON encoded Protobuf payload.

The client MAY gzip the content and in that case SHOULD include
The client MAY gzip the content and in that case MUST include
"Content-Encoding: gzip" request header. The client MAY include
"Accept-Encoding: gzip" request header if it can receive gzip-encoded responses.

Expand Down

0 comments on commit 90245b8

Please sign in to comment.