Skip to content

Conversation

JoshFerge
Copy link
Member

this is causing traces that are far too large to view, and doesn't really help for debugging. let's not propagate traces here

@JoshFerge JoshFerge requested a review from a team as a code owner October 8, 2025 19:25
@JoshFerge JoshFerge requested review from a team October 8, 2025 19:25
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 8, 2025
@JoshFerge JoshFerge requested a review from a team October 8, 2025 19:26
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #101189   +/-   ##
========================================
  Coverage   81.08%    81.08%           
========================================
  Files        8672      8672           
  Lines      384683    384683           
  Branches    24280     24280           
========================================
+ Hits       311917    311929   +12     
+ Misses      72419     72407   -12     
  Partials      347       347           

Updates test_dispatch_task_with_occurrence to expect the headers parameter
added in the previous commit for disabling trace propagation.
"project_id": project_id,
"eventstream_type": eventstream_type,
},
headers={"sentry-propagate-traces": False},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Header Value Type Mismatch Causes Tracing Issues

The sentry-propagate-traces header uses a boolean False value. Headers typically expect strings, which could lead to serialization or processing issues in the tracing system.

Fix in Cursor Fix in Web

@JoshFerge JoshFerge merged commit c6388d0 into master Oct 8, 2025
64 checks passed
@JoshFerge JoshFerge deleted the jferg/dont-prop-trace branch October 8, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants