- Notifications
You must be signed in to change notification settings - Fork3.2k
Add multi-user HTTP mode: per-request GitHub token, docs, and tests#489
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
- Replace per-request MCP server creation with token-aware client factories- Add proper HTTP timeouts and graceful shutdown- Enhance token validation with strict Bearer format checking- Add comprehensive tests for token extraction and context injection- Improve error handling and response formatting- Update documentation to reflect performance optimizations
SamMorrowDrums commentedJun 7, 2025
Hi thank you for your contribution, we will be releasing a remote server soon, using this repo as a library. We won't be merging PRs adding this functionality for the foreseeable future. |
thomcost commentedJun 8, 2025
Who is "we"? |
SamMorrowDrums commentedJun 8, 2025
GitHub, I am an internal maintainer. |
tcostello-godaddy commentedJun 8, 2025
Do you have any ballpark dates? please :) |
SamMorrowDrums commentedJun 8, 2025
I cannot be exact, but its fairly imminent. |
tonytrg commentedJun 16, 2025
@tcostello-godaddy remote mcp is live :)https://github.com/github/github-mcp-server?tab=readme-ov-file#remote-github-mcp-server @thomcost thank you for the contribution, since we offer remote mcp now, as@SamMorrowDrums mentioned for now this will not be added to the local version. |
SamMorrowDrums commentedJun 16, 2025
Update: we may reconsider to unblock GHES use-cases. |
tonytrg commentedJun 17, 2025
related to#523 |
ransonjb commentedJun 19, 2025
Hey@SamMorrowDrums |
SamMorrowDrums commentedJun 19, 2025
Hard to say@ransonjb at this moment I am afraid. There are a lot of implementation details for anything beyond single server hosting of MCP too, and I am also cautious about hosting support burdon too. Things like concurrency, authentication etc. On our remote server we have to do a lot to handle horizontal scaling, safe concurrency and we authorize users on the way in (before hitting the MCP server at all), and we have full oauth support. These sorts of considerations make it difficult for end users build and release it safely without customisation for their individual hosting scenarios. Using a fork can unblock you if you need this right away. I expect at some point we will merge the streamable http transport, but I cannot say when. |
icep87 commentedJul 23, 2025
Any progress on the unblocking of GHES or adding HTTP-stream support for this? |
rupinder2 commentedAug 28, 2025
Just testing out this branch, and I can't seem to connect using MCP inspector. |
Closes: Issue#471