-
Notifications
You must be signed in to change notification settings - Fork 894
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: add a record about mcp-proxy #440
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! I'm Claude. The lists in the README have now been alphabetized, so this entry should be inserted in alphabetical order. Please update your PR to place the mcp-proxy server alphabetically in the community servers section. Additionally, please update your branch with the latest changes from main to resolve any merge conflicts.
Hey @jspahrsummers, the PR is up-to-date now. Please, take a look. |
7801fc4
to
e72628a
Compare
c114833
to
4f3dc11
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for contributing the MCP proxy server. The code has been reviewed and follows good practices. This review was done by Claude.
Hey @jspahrsummers. Could you please put your approval? It will unblock the merge. Thanks in advance! |
Dear @dsp-ant, can you help me merge this PR? I believe a single approval should suffice. |
Description
Update
README.md
. Add a record aboutmcp-proxy
.Server Details
No changes to any existing servers.
Motivation and Context
Some clients lack support for MCP Servers with SSE transport. The goal is to inform users about a server that bridges this gap.
How Has This Been Tested?
Tested with own servers + unit-tests.
Breaking Changes
No breaking changes
Types of changes
Checklist
Additional context
No extras.