-
-
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
Enrich Transactions with Context Data #1469
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1469 +/- ##
============================================
- Coverage 75.86% 75.75% -0.11%
- Complexity 1922 1926 +4
============================================
Files 190 190
Lines 6555 6580 +25
Branches 659 665 +6
============================================
+ Hits 4973 4985 +12
- Misses 1263 1270 +7
- Partials 319 325 +6
Continue to review full report at Codecov.
|
@denrase left a few suggestions but looks good to me. |
`Instructions and example for changelog`$
To the changelog entry, please add a link to this PR (consider a more descriptive message):` - Enrich Transactions with Context Data(#1469)
|
📜 Description
OutboxSender
picks up a file containing a transaction item, it will callcaptureTransaction
instead of packing it up and calling 'captureEnvelope'.contexts.trace.sampled
to true manually as a workaround, as this is a transient property and not serialized. Without it, the transaction would be dropped.💡 Motivation and Context
💚 How did you test it?
📝 Checklist
🔮 Next steps