Skip to content

[Backport v7.x] fix: honor backpressure in decompression interceptor - #5837

Open
mcollina wants to merge 1 commit into
v7.xfrom
backport-5829-to-v7.x
Open

mcollina wants to merge 1 commit into
v7.xfrom
backport-5829-to-v7.x

Conversation

@mcollina

Copy link
Copy Markdown
Member

Backports #5829 to v7.x.

This also includes the stable RetryController proxy required by the decompression backpressure fix because v7.x does not contain #5405.

* fix: honor decompression backpressure

Signed-off-by: Matteo Collina <hello@matteocollina.com>

* fix(decompress): disable size limit by default

Signed-off-by: Matteo Collina <hello@matteocollina.com>

---------

Signed-off-by: Matteo Collina <hello@matteocollina.com>
(cherry picked from commit 19901d8)
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.30769% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.11%. Comparing base (a7914e5) to head (ef01396).

Files with missing lines Patch % Lines
lib/interceptor/decompress.js 90.35% 22 Missing ⚠️
lib/handler/retry-handler.js 98.59% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##             v7.x    #5837    +/-   ##
========================================
  Coverage   93.10%   93.11%            
========================================
  Files         112      112            
  Lines       37145    37356   +211     
========================================
+ Hits        34584    34784   +200     
- Misses       2561     2572    +11     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mcollina
mcollina requested review from metcoder95 and ronag and removed request for ronag September 17, 2026 07:52
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