Skip to content

fix(node): fix http.request for minipass-fetch 2.x - #2004

Merged
kt3k merged 2 commits into
denoland:mainfrom
kt3k:fix/fix-for-minipass-fetch-2
Mar 8, 2022
Merged

kt3k merged 2 commits into
denoland:mainfrom
kt3k:fix/fix-for-minipass-fetch-2

Conversation

@kt3k

@kt3k kt3k commented Mar 7, 2022

Copy link
Copy Markdown
Contributor

This PR fixes the call of http.request without href option.

minipass-fetch@>2 calls http.request without href option (this commit started it).

npm started depending on minipass-fetch@2 indirectly at this commit, and that broke the integration test of npm install in compat mode in CI

closes #2001

@kt3k
kt3k requested a review from bartlomieju as a code owner March 7, 2022 10:39

@bartlomieju bartlomieju left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, nice find

@kt3k
kt3k merged commit a7872e5 into denoland:main Mar 8, 2022
@kt3k
kt3k deleted the fix/fix-for-minipass-fetch-2 branch March 8, 2022 05:18
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.

compat: npm@8.5.3 doesn't work with compat mode

2 participants