Skip to content

fix: validate envelope item payload lengths before allocating a read buffer - #5541

Merged
jamescrosswell merged 1 commit into
getsentry:mainfrom
thaildhe172591:fix/5536-envelope-payload-length-bound
Sep 8, 2026
Merged

jamescrosswell merged 1 commit into
getsentry:mainfrom
thaildhe172591:fix/5536-envelope-payload-length-bound

fix: bound the envelope item payload length before allocating a read …

1362295
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 8, 2026 in 0s

74.74% (+0.03%) compared to be2a785

View this Pull Request on Codecov

74.74% (+0.03%) compared to be2a785

Details

Codecov Report

❌ Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.74%. Comparing base (be2a785) to head (1362295).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
src/Sentry/Protocol/Envelopes/EnvelopeItem.cs 91.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5541      +/-   ##
==========================================
+ Coverage   74.70%   74.74%   +0.03%     
==========================================
  Files         515      515              
  Lines       18948    18956       +8     
  Branches     3696     3694       -2     
==========================================
+ Hits        14155    14168      +13     
+ Misses       3909     3908       -1     
+ Partials      884      880       -4     

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