Skip to content

fstree: limit ranged payload stream from object start#4046

Merged
cthulhu-rider merged 1 commit into
masterfrom
fix-range-split
Jun 26, 2026
Merged

fstree: limit ranged payload stream from object start#4046
cthulhu-rider merged 1 commit into
masterfrom
fix-range-split

Conversation

@End-rey

@End-rey End-rey commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 26.78%. Comparing base (b163f5b) to head (f5a4361).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4046      +/-   ##
==========================================
+ Coverage   26.76%   26.78%   +0.01%     
==========================================
  Files         688      688              
  Lines       46674    46674              
==========================================
+ Hits        12493    12501       +8     
+ Misses      33049    33041       -8     
  Partials     1132     1132              

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

Comment thread pkg/local_object_storage/blobstor/fstree/fstree_test.go Outdated
FSTree range reads returned the raw payload stream when the requested range
started at offset zero and no payload bytes were buffered in the prefix. This
could make ranged GET send more bytes than requested and trigger client-side
payload size overflow.

Signed-off-by: Andrey Butusov <andrey@nspcc.io>
@cthulhu-rider cthulhu-rider merged commit 0af16a2 into master Jun 26, 2026
22 checks passed
@cthulhu-rider cthulhu-rider deleted the fix-range-split branch June 26, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants