Name HTTPX request spans according to convention

[vrslev]

Describe your environment

~

What happened?

~

Steps to Reproduce

~

Expected Result

~

Actual Result

~

Additional context

Recent OpenTelemetry article recommends naming HTTP spans this way: GET /api/users/:id. Currently, HTTPX spans include only the method (GET), which is inconsistent with other instrumentations such as Litestar or FastAPI.

Would you like to implement a fix?

None

Tip

_{React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.}

[rads-1996]

I can work on this, can I please be assigned this issue. Thanks!

[vrslev]

Hi, great! Go ahead :)

[rads-1996]

Hi, great! Go ahead :)

If possible can you please assign the issue to me.

[rads-1996]

@vrslev It looks like the current behavior for httpx returning only GET is correct as it creates client-side spans, you can refer to this PR - #1759, which implemented the specification as per which -

So fastapi being a server-side instrumentation correctly creates spans with {method} and {route} whereas httpx creates spans with {http.method}.

[vrslev]

Oh, that’s too bad. Also found this: open-telemetry/semantic-conventions#923. Thanks for the enthusiasm anyway!