Change network.transport from recommended to opt-in in the HTTP semconv#402
Conversation
587e614 to
0b9c69e
Compare
mateuszrzeszutek
left a comment
There was a problem hiding this comment.
👍
It was pretty much always skipped anyway:
If not default (
tcpforHTTP/1.1andHTTP/2,udpforHTTP/3).
Actually this is not true. In Node.js scenarios I think it is not entirely unheard of to communicate with a local HTTP server over Unix domain sockets. We have seen this in the wild with our agents I believe |
Looking for
I agree it's a valid concern, but having this attribute outside of HTTP spec and effectively opt-in still works. |
network.transport from HTTP semconv, effectively making it opt-innetwork.transport from recommended to opt-in in the HTTP semconv
…conv (open-telemetry#402) Co-authored-by: Joao Grassi <[email protected]>
Fixes #389
network.transportis typically derivable fromnetwork.protocol.version:tcpudpso it doesn't provide much value to capture in HTTP semconv.
Changes
Change
network.transportfrom recommended to opt-in in the HTTP semconv.Merge requirement checklist
schema-next.yaml updated with changes to existing conventions.