Skip to content

fix(profiling): release the EventPipe session when the SDK shuts down - #5470

Merged
jamescrosswell merged 13 commits into
mainfrom
fix/5418-profiler-session-dispose
Aug 26, 2026
Merged

jamescrosswell merged 13 commits into
mainfrom
fix/5418-profiler-session-dispose

ref(profiling): guard Start() after disposal and name the session han…

5817d9e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 26, 2026 in 0s

74.74% (-0.13%) compared to 2be688c

View this Pull Request on Codecov

74.74% (-0.13%) compared to 2be688c

Details

Codecov Report

❌ Patch coverage is 68.35443% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.74%. Comparing base (2be688c) to head (5817d9e).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...ry.Profiling/SamplingTransactionProfilerFactory.cs 65.00% 10 Missing and 4 partials ⚠️
src/Sentry.Profiling/SampleProfilerSession.cs 35.29% 7 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5470      +/-   ##
==========================================
- Coverage   74.87%   74.74%   -0.13%     
==========================================
  Files         513      513              
  Lines       18659    18829     +170     
  Branches     3636     3682      +46     
==========================================
+ Hits        13970    14074     +104     
- Misses       3819     3875      +56     
- Partials      870      880      +10     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.