Skip to content

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

Draft
jamescrosswell wants to merge 1 commit into
mainfrom
fix/5418-profiler-session-dispose
Draft

fix(profiling): release the EventPipe session when the SDK shuts down#5470
jamescrosswell wants to merge 1 commit into
mainfrom
fix/5418-profiler-session-dispose

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

948f8b5
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 6, 2026 in 0s

74.74% (+0.03%) compared to e669940

View this Pull Request on Codecov

74.74% (+0.03%) compared to e669940

Details

Codecov Report

❌ Patch coverage is 65.78947% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.74%. Comparing base (e669940) to head (948f8b5).

Files with missing lines Patch % Lines
src/Sentry.Profiling/SampleProfilerSession.cs 33.33% 6 Missing and 2 partials ⚠️
...ry.Profiling/SamplingTransactionProfilerFactory.cs 73.68% 4 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5470      +/-   ##
==========================================
+ Coverage   74.71%   74.74%   +0.03%     
==========================================
  Files         513      513              
  Lines       18729    18758      +29     
  Branches     3663     3669       +6     
==========================================
+ Hits        13993    14021      +28     
+ Misses       3865     3858       -7     
- Partials      871      879       +8     

☔ 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.