Add http oauth support #1
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This adds HTTP server mode to the GitHub MCP Server, enabling multi-client support with "bring your own token" OAuth functionality. This is useful for enterprise scenarios where a single MCP server instance handles multiple external clients, each authenticating with their own credentials. Key changes: - Add `http` command to start HTTP server on configurable port (default 8080) - Support per-request OAuth tokens via Authorization header - Fall back to GITHUB_PERSONAL_ACCESS_TOKEN env var if no header provided - Modify client factories to extract token from request context - Add comprehensive HTTP server documentation to README This implementation is inspired by PR github#888 by @Dreadnoth, updated to work with the current codebase architecture and dependencies. Co-authored-by: Dreadnoth <[email protected]> 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Co-authored-by: Copilot <[email protected]>
* Add "trust" option to MCP server configuration for bypassing confirmations * Additional configuration * Add co-author Co-authored-by: Michael Vorburger <[email protected]> --------- Co-authored-by: Michael Vorburger <[email protected]>
* adding better response * adding check for subdomain isolation and return raw resp for better better llm response * adding subdomain for uploads too * remove unnecessary comments * better error message * fix linter
* Add additional toolsets in remote mcp * Add copilot * Fix urls * Move copilot and spaces toolsets to remote-only section --------- Co-authored-by: LuluBeatson <[email protected]>
* docs: add github_support_docs_search to REMOTE server * add copilot_spaces * fix typos * Add support tool to remote toolsets table
* add toolset default to make configuration easier * fix readme * adding transformer to cleanly handle special toolsets * cleaning code * fixing cli message * remove duplicated test * Update internal/ghmcp/server.go Co-authored-by: Copilot <[email protected]> * Update internal/ghmcp/server_test.go Co-authored-by: Copilot <[email protected]> * adding error message for invalid toolsets * fix merge conflict * add better formatting --------- Co-authored-by: Copilot <[email protected]>
Co-authored-by: Julia <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes: