Opentelemetry-specification: Remove semantic convention for http.status_text

Created on 15 Sep 2020  路  4Comments  路  Source: open-telemetry/opentelemetry-specification

HTTP semantic convention defines http.status_text as HTTP reason phrase. But even RFC sounds like that reason phrase is an obsolete artefact (emphasis is mine):

The reason-phrase element exists for the sole purpose of providing a
textual description associated with the numeric status code, mostly
out of deference to earlier Internet application
protocols that were
more frequently used with interactive text clients. A client SHOULD
ignore
the reason-phrase content

Should we drop this semantic attribute as serving no actual purpose?

semantic-conventions after-ga trace

Most helpful comment

We could also consider marking the attribute as deprecated (should not be set by instrumentation, but if you get it, this is the meaning).

I'd rather remove it (unless OC really needs it for backwards compatibility).

All 4 comments

We inherited this attribute from OpenCensus AFAIK. But if the RFC says it SHOULD be ignored then I think there would be a point in doing so. I think it also does not exist from HTTP 2 on.

We could also consider marking the attribute as deprecated (should not be set by instrumentation, but if you get it, this is the meaning).

We could also consider marking the attribute as deprecated (should not be set by instrumentation, but if you get it, this is the meaning).

I'd rather remove it (unless OC really needs it for backwards compatibility).

+1 to remove this

Done in #972.

Was this page helpful?
0 / 5 - 0 ratings