Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

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

Open
thomcost wants to merge2 commits intogithub:main
base:main
Choose a base branch
Loading
fromthomcost:multi-user-http

Conversation

thomcost
Copy link

 - Added multi-user HTTP mode with per-request GitHub token via Authorization header. - Updated documentation and added unit/e2e tests. - All tests passing.

Closes: Issue#471

@thomcostthomcost requested a review froma team as acode ownerJune 7, 2025 07:46
- 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
Copy link
Collaborator

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
Copy link
Author

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.

Who is "we"?

tcostello-godaddy reacted with laugh emoji

@SamMorrowDrums
Copy link
Collaborator

GitHub, I am an internal maintainer.

tcostello-godaddy reacted with thumbs up emoji

@tcostello-godaddy
Copy link

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.

Do you have any ballpark dates? please :)

@SamMorrowDrums
Copy link
Collaborator

I cannot be exact, but its fairly imminent.

@tonytrg
Copy link
Contributor

@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.
Still the contribution is really appreciated, cheers!

@SamMorrowDrums
Copy link
Collaborator

Update: we may reconsider to unblock GHES use-cases.

natlight reacted with thumbs up emoji

@tonytrg
Copy link
Contributor

related to#523

@ransonjb
Copy link

Update: we may reconsider to unblock GHES use-cases.

Hey@SamMorrowDrums
I would add a "+1": we are working on an important project where support for GHES is required.
Would you have an ETA for this support?

@SamMorrowDrums
Copy link
Collaborator

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.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

5 participants
@thomcost@SamMorrowDrums@tcostello-godaddy@tonytrg@ransonjb
[8]ページ先頭
©2009-2025 Movatter.jp