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 and pc-ahrens 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?

icep87 and pc-ahrens reacted with thumbs up emoji

@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.

pc-ahrens reacted with thumbs up emoji

@icep87
Copy link

Any progress on the unblocking of GHES or adding HTTP-stream support for this?

pc-ahrens reacted with thumbs up emoji

@rupinder2
Copy link

Just testing out this branch, and I can't seem to connect using MCP inspector.

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.

7 participants

@thomcost@SamMorrowDrums@tcostello-godaddy@tonytrg@ransonjb@icep87@rupinder2

[8]ページ先頭

©2009-2025 Movatter.jp