-
Notifications
You must be signed in to change notification settings - Fork 992
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support HTTP request/response size #880
Comments
Thanks. Surprised we didn't think of that earlier. Will do it with a Just requires a change to |
Just out of curiosity, would this include body-only-size or include the headers as well? |
@mweirauch I think we should do both body and headers (with a tag?) as much as possible. Also, a separate metric for request and response sizes would be potentially useful. |
Sounds good. |
@mweirauch I guess you meant |
Eek, things go wrong when doing stuff in between. What I meant was ... |
Hi is there any updates (after 2 years)? |
Hi is there any updates (after 3 years)? |
hi, is there any updates (after 4 years)? |
There was:
It's not a guarantee that it will make it to 1.11 but issues like this ( |
Hi @shakuzen , |
From a Framework instrumentation perspective: An easy way to instrument this would be to read the Also, even if this first problem is solved, the message size at the web Framework level is not the same as the actual message size on the network. Servers and Servlet containers in general support message compression and perform this task on-the-fly. This means that this metric instrumentation would need to happen at the server level, not at the web framework level. This part is highlighted in the OTel semantic conventions as well and the metric itself is optional. To summarize, I think that mentioning I think the best way to make progress on this is to review the available binders for HTTP servers and clients, and figure out if this type of instrumentation is possible in an efficient manner for each. |
Please add support to get http response size as a metric value
The text was updated successfully, but these errors were encountered: