Skip to content

Conversation

@ojacques
Copy link

No description provided.

@harshh-jainn
Copy link

@james-pplx - can we please get this merged; very useful to have a web proxy

@gurvancampion
Copy link

@kesku Any chance we could get this merged? Currently at work, I cannot use it due to the corporate proxy

@kesku
Copy link
Collaborator

kesku commented Nov 8, 2025

Hey @gurvancampion — unfortunately this PR is quite a bit out of date now. The MCP has gone through multiple updates since it was opened, and the web proxy implementation here conflicts with several newer changes. We’d need a fresh implementation based on the current codebase to move this forward.

One other consideration is that we’re planning to migrate to an HTTP-based transport layer soon, which may break or complicate the current STDIO-based setup — so adding a proxy right now could introduce additional issues into the codebase.

Regardless, I'll look into getting something like this merged soon.

@gurvancampion
Copy link

Hey @gurvancampion — unfortunately this PR is quite a bit out of date now. The MCP has gone through multiple updates since it was opened, and the web proxy implementation here conflicts with several newer changes. We’d need a fresh implementation based on the current codebase to move this forward.

One other consideration is that we’re planning to migrate to an HTTP-based transport layer soon, which may break or complicate the current STDIO-based setup — so adding a proxy right now could introduce additional issues into the codebase.

Regardless, I'll look into getting something like this merged soon.

Thanks for the quick answer. Let me know if I can help. I could push a new MR based on the existing codebase, or wait till you have migrated to re-check at that time

@ojacques
Copy link
Author

ojacques commented Nov 8, 2025

I'll look at it.

@kesku kesku mentioned this pull request Nov 9, 2025
@kesku
Copy link
Collaborator

kesku commented Nov 9, 2025

Just an update here

I’ve implemented proxy support in #66, which is now merged into main.

This is a temporary solution since as stated previously we'll be migrating to streamable HTTP transport soon. Shouldn’t affect end users in the meantime!

Thanks again @ojacques and @gurvancampion for flagging this!

@kesku kesku closed this Nov 9, 2025
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.

4 participants