-
-
Notifications
You must be signed in to change notification settings - Fork 446
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
Feat: Add option to filter which origins receive tracing headers #1698
Conversation
I realised we also need add option to set this property from the external configuration. |
Codecov Report
@@ Coverage Diff @@
## main #1698 +/- ##
============================================
+ Coverage 74.80% 74.83% +0.03%
- Complexity 2077 2086 +9
============================================
Files 210 211 +1
Lines 7430 7451 +21
Branches 782 788 +6
============================================
+ Hits 5558 5576 +18
- Misses 1491 1494 +3
Partials 381 381
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
📜 Description
Add
tracingOrigins
toSentryOptions
and respect it in OkHttp, WebClient and RestTemplate integrations.💡 Motivation and Context
Fixes #1697
Fixes #1585
💚 How did you test it?
Integration tests.
📝 Checklist