getsentry / getsentry/sentry-javascript
Extension of shouldCreateSpanForRequest to cover usual cases
Open
Improvement
Node.js
- Dominant language
- TypeScript
- Stars
- 8.7k
- Forks
- 1.8k
- Avg merge
- 1d 17h
- Merged PRs (30d)
- 515
Description
### Problem Statement
Right now the `shouldCreateSpanForRequest` callback could be helpful to distinguish between API endpoints and decide whether a request should be decorated with the distributed tracing headers. Besides just having the URL at hand to make a decision, it would be helpful to provide some more request specific information – especially the HTTP method. That would help to only send the tracing headers to API methods that could deal with those.
### Solution Brainstorm
Extend the `shouldCreateSpanForRequest` by an additional parameter providing at least the HTTP method.
Contributor guide
Assessment
This issue has not been assessed yet.