-
Notifications
You must be signed in to change notification settings - Fork 3k
Add a Remote MCP configuration example that employs a PAT #514
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
Changes from 4 commits
c1e61b8
27b82d6
ef8f612
4a1cb43
04856da
c7ff79b
ad1df84
d72a937
adef317
a86f3dd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -30,21 +30,41 @@ For quick installation, use one of the one-click install buttons above. Once you | |||||
|
|
||||||
| ### Usage in other MCP Hosts | ||||||
|
|
||||||
| For MCP Hosts that have been [configured to use the remote GitHub MCP Server](docs/host-integration.md), add the following JSON block to the host's configuration: | ||||||
| For MCP Hosts that have been [configured to use the remote GitHub MCP Server](docs/host-integration.md), choose the appropriate JSON block from the examples below and add it to the host's configuration: | ||||||
|
|
||||||
| <table> | ||||||
| <tr><th>Using OAuth</th><th>Using a GitHub PAT</th></tr> | ||||||
| <tr valign='top'> | ||||||
| <td> | ||||||
|
|
||||||
| ```json | ||||||
| { | ||||||
| "mcp": { | ||||||
| "servers": { | ||||||
| "github": { | ||||||
| "type": "http", | ||||||
| "url": "https://api.githubcopilot.com/mcp/" | ||||||
| } | ||||||
| "mcpServers": { | ||||||
| "github": { | ||||||
| "url": "https://api.githubcopilot.com/mcp/" | ||||||
| } | ||||||
| } | ||||||
| } | ||||||
| ``` | ||||||
|
|
||||||
| </td> | ||||||
| <td> | ||||||
|
|
||||||
| ```json | ||||||
| { | ||||||
| "mcpServers": { | ||||||
| "github": { | ||||||
| "url": "https://api.githubcopilot.com/mcp/", | ||||||
| "authorization_token": "<your GitHub PAT>" | ||||||
|
||||||
| "authorization_token": "<your GitHub PAT>" | |
| "authorization_token": "bearer <your GitHub PAT>" |
Technically it needs that. Claude code adds it automatically apparently.
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.
Hmmm ... none of their examples include Bearer and this wasn't even meant to be Claude-specific.
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.
(see disclaimer below)
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.
Their tools add it automatically but it still works with it included and I'm worried about other clients doing it wrong. I confirmed that with Anthropic yesterday.
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.
In the Claude docs, I don't see a way to prompt or otherwise read the PAT dynamically (e.g. from an environment variable). As far as I can tell, it has to be pasted into the JSON as plain text.
Please correct me if I'm wrong!
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.
I believe you are correct