Skip to content

feat: use standard fetch API when using node >=18#17269

Open
CamilleLetavernier wants to merge 1 commit intomasterfrom
issues/12775
Open

feat: use standard fetch API when using node >=18#17269
CamilleLetavernier wants to merge 1 commit intomasterfrom
issues/12775

Conversation

@CamilleLetavernier
Copy link
Copy Markdown
Contributor

fixes #12775

What it does

  • remove the dependency to 'http(s)-proxy-agent'
  • replace it with Node fetch + undici (for Proxy)
  • replace the deprecated parseurl() with new URL()

How to test

  • Test some commands that depend on the NodeRequestService / Proxy, e.g.:
    • In Command Palette, check that "Configure Display Language" works properly
    • In Extensions, search for any OpenVSX extension and verify that extensions can be found/downloaded

Follow-ups

Breaking changes

  • This PR introduces breaking changes and requires careful review. If yes, the breaking changes section in the changelog has been updated.

Attribution

Review checklist

Reminder for reviewers

@github-project-automation github-project-automation bot moved this to Waiting on reviewers in PR Backlog Mar 31, 2026
@ndoschek ndoschek requested a review from eneufeld April 9, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting on reviewers

Development

Successfully merging this pull request may close these issues.

use standard fetch API when using node >=18

1 participant