Skip to content

Conversation

phoval
Copy link
Contributor

@phoval phoval commented Jul 27, 2025

Fix #124 - add proxy environment support using undici

Copy link

pkg-pr-new bot commented Aug 27, 2025

Open in StackBlitz

npx https://pkg.pr.new/mcp-remote@131

commit: 782ed64

@geelen
Copy link
Owner

geelen commented Aug 27, 2025

I'm a little hesitant to introduce this by default for everybody, not because it's not a good idea, just because my capacity to test stuff at the moment is pretty low. So what I'll do is get you and @rohanpower to test it using npx https://pkg.pr.new/mcp-remote@131?

I think I could merge this right away if the undici injection was conditional, only activating if a --enable-proxy flag was set? In future it might make sense to just have this in all the time but I don't want to risk breaking it for anyone when I don't have a lot of time to fix it.

@geelen geelen mentioned this pull request Aug 27, 2025
@geelen geelen merged commit 338df58 into geelen:main Aug 27, 2025
3 checks passed
@geelen
Copy link
Owner

geelen commented Aug 27, 2025

Thanks! Released as [email protected]

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.

Add proxy support
2 participants