Skip to content

fix: harden pinned http requests without undici - #432

Merged
jonluca merged 1 commit into
APIDevTools:mainfrom
frodeaa:remove-undici-dep
Oct 11, 2026
Merged

jonluca merged 1 commit into
APIDevTools:mainfrom
frodeaa:remove-undici-dep

Conversation

@frodeaa

@frodeaa frodeaa commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Drop undici to cut a 2.3 MB dependency, using node:http/https with the
same DNS-pinned lookup

Rel: e8190c1

Drop undici to cut a 2.3 MB dependency, using node:http/https
with the same DNS-pinned lookup
@jonluca
jonluca enabled auto-merge (squash) October 11, 2026 19:05
@jonluca
jonluca merged commit 8c4946a into APIDevTools:main Oct 11, 2026
11 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 38144513849

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage increased (+0.2%) to 89.49%

Details

  • Coverage increased (+0.2%) from the base build.
  • Patch coverage: 2 uncovered changes across 1 file (81 of 83 lines covered, 97.59%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
lib/resolvers/http.ts 83 81 97.59%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1761
Covered Lines: 1614
Line Coverage: 91.65%
Relevant Branches: 1512
Covered Branches: 1315
Branch Coverage: 86.97%
Branches in Coverage %: Yes
Coverage Strength: 160475.75 hits per line

💛 - Coveralls

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