-
Notifications
You must be signed in to change notification settings - Fork 175
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
Capture request and response bodies #857
Comments
Related open-telemetry/opentelemetry-specification#1061 (comment) EDIT: But here it may make sense to keep in one issue to define if we even want to support this at all. It seems problematic, both from a performance and also privacy perspective. |
This is an opt-in feature not enabled by default only users who need this would enable it with all the implications of course. |
This would be very useful for certain scenarios |
Hello please has there been any progress made here. This specification would be really helpful in many enterprise Environments for also in many app dev scenarios for distributed trace of app data flow validations |
upvoted this request :-) would be really usefull |
+1 |
@pavolloffay Is this still a priority for you ? Were there any discussions within the Otel team ? How can we take this forward ? I would really like to make this happen. @Oberon00 |
I haven't looked into this ticket since October 2020 |
@gmreads check out open-telemetry/oteps#219 |
I've been looking into this, and read through open-telemetry/oteps#219 . Today, we patch the instrumentation for our clients so we can get the request body but the main problem is that there's no standard for this. I propose we standardize the semantic attribute name of Would love to start working on this. |
Some context: open-telemetry/opentelemetry-specification#2888 proposed extending attributes to support complex types like maps and heterogenous arrays. If added, this would be a natural place to put http request / response bodies. Unfortunately, we didn't achieve consensus and the PR is currently closed. That effort could be restarted, but I think it would take new use cases and / or new arguments to move the needle. |
Open a new PR for that open-telemetry/oteps#234 |
(I asked this on oteps 234 as well) Should the payload be a span event instead of a span attribute? Most backends have higher size limits for span events. |
That doesn't make sense from a semantics point of view though... a piece of request/response data is not "an event" IMHO. And the HTTP span itself already represents a request or response, so there is no room to define a "send event" inside of that in my mind. |
I would prefer to have body and other request/response attributes as a structured log. Moreover, to have the ability to separate them into different pipelines from other logs (different security and rotation policies). It is make sense to me to be able to select them by certain parameters and groups and able to process via anomaly detection tools. |
What are you trying to achieve?
I want to capture request and response bodies. The content can be used for business transaction monitoring and security.
To move this forward we need:
http.request.body
/http.response.body
or be transport agnostic e.g.request.body
,response.body
.OTEL_INSTRUMENTATION_CAPTURE_REQUEST_BODY=bool
Additional context.
The text was updated successfully, but these errors were encountered: