Skip to content
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

Collect RabbitMQ message headers #6293

Closed
drewhammond opened this issue Jul 8, 2022 · 1 comment · Fixed by #6454
Closed

Collect RabbitMQ message headers #6293

drewhammond opened this issue Jul 8, 2022 · 1 comment · Fixed by #6454
Labels
contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome enhancement New feature or request

Comments

@drewhammond
Copy link

Is your feature request related to a problem? Please describe.

We store valuable identifiers in RabbitMQ (AMQP 0-9-1) message headers. In Elastic APM, these headers are collected as attributes in spans, but in OpenTelemetry they are not.

Describe the solution you'd like

We'd like the ability to toggle the collection of these message headers so that they can be included in spans. As a nice-to-have, we'd like the list of collected headers to be configurable.

Describe alternatives you've considered
We have not found an attractive alternative with only using automatic instrumentation.

@drewhammond drewhammond added the enhancement New feature or request label Jul 8, 2022
@mateuszrzeszutek mateuszrzeszutek added the contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome label Jul 11, 2022
@mateuszrzeszutek
Copy link
Member

Looks really similar to: #5238
I guess it could be implemented as a generic messaging headers-extracting-thing (perhaps similar to how we're extracting HTTP headers)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants