feat(fetch): add proxy support via standard proxy environment variables#417
feat(fetch): add proxy support via standard proxy environment variables#417mact24 wants to merge 1 commit intojackwener:mainfrom
Conversation
…nv vars Use undici ProxyAgent to route Node-side fetch calls through the system proxy when any of the standard proxy environment variables are set. Browser-side fetch paths are unaffected. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Good idea — proxy support for Chinese users is a real need. Found some issues with the current implementation: CriticalSOCKS5 will crash at runtime undici Proxy selection ignores request scheme
Recommended fix: Replace the manual proxy resolution with undici's built-in WarningsNode version conflict — ProxyAgent created per-request — Only covers YAML pipeline fetch — 10+ TS adapters call Node.js Minor
|
Summary
fetchpipeline step (used by allstrategy: publicadapters like HackerNews, Wikipedia, arXiv, etc.) previously called Node.js nativefetchdirectly, which ignores system proxy settingsundiciProxyAgent, reading standard proxy environment variables at runtimeHow it works
When any of the following env vars are set, Node-side
fetchcalls are routed through the proxy. Browser-side fetch paths are unaffected (Chrome uses its own proxy settings).