Skip to content

fix(router): avoid subscription clients on subgraphs with no subscriptions - #3094

Draft
dkorittki wants to merge 1 commit into
mainfrom
dominik/router-19-high-number-of-goroutines-for-ws-pingloop
Draft

fix(router): avoid subscription clients on subgraphs with no subscriptions#3094
dkorittki wants to merge 1 commit into
mainfrom
dominik/router-19-high-number-of-goroutines-for-ws-pingloop

fix: conditionally create subscription client

b6ebdb1
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 1, 2026 in 0s

39.37% (target 30.00%)

View this Pull Request on Codecov

39.37% (target 30.00%)

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.37%. Comparing base (3c15243) to head (b6ebdb1).
⚠️ Report is 49 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3094       +/-   ##
===========================================
- Coverage   71.04%   39.37%   -31.68%     
===========================================
  Files         335       14      -321     
  Lines       49334     1651    -47683     
  Branches     6033        0     -6033     
===========================================
- Hits        35047      650    -34397     
+ Misses      14261      873    -13388     
- Partials       26      128      +102     

see 349 files with indirect coverage changes

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