-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Backport of support for http.request.method
in OTel span processor
#11926
Labels
Comments
Do you mind opening a PR for this? I'll happily approve and merge it for you! |
Will do, no worries 🙂 |
2 tasks
Fixed |
mydea
pushed a commit
that referenced
this issue
May 7, 2024
This will go out with the next backport release, thanks again! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there an existing issue for this?
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/node
SDK Version
latest
Framework Version
n/a
Link to Sentry event
n/a
Issue
Please also support
http.request.method
instead of onlyhttp.method
in v7.Fix: https://github.com/getsentry/sentry-javascript/blob/develop/packages/opentelemetry/src/utils/parseSpanDescription.ts#L41
Old code: https://github.com/getsentry/sentry-javascript/blob/v7/packages/opentelemetry-node/src/utils/parseOtelSpanDescription.ts#L27
Should take no more than a couple of minutes 🙂
The text was updated successfully, but these errors were encountered: