-
Notifications
You must be signed in to change notification settings - Fork 581
test: Add test for get_baggage with DSN
#5441
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
Conversation
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. This PR will not appear in the changelog. 🤖 This preview updates automatically when you update the PR. |
Codecov Results 📊✅ 13 passed | Total: 13 | Pass Rate: 100% | Execution Time: 7.58s All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 13657 uncovered lines. Files with missing lines (180)
Generated by Codecov Action |
93cc2c4 to
c204b78
Compare
This adds a test for
get_baggagethat demonstrates that the public key form the DSN is included in the resulting baggage string. This came up internally while debugging an incomplete baggage header in the Sentry backend.(Turns out the root cause was not 100% related to this test but we might wanna keep it anyway)