Skip to content

Add HTTP+SSE server command #47

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Add HTTP+SSE server command #47

wants to merge 1 commit into from
+160 −7

Conversation

iancanderson
Copy link

I'm not sure we want to merge this, but this allowed me to try dotcom chat as an MCP host in a dev environment, using mark3labs/mcp-go in copilot-api, connecting to a localhost version of github-mcp-server via HTTP/SSE.

export GITHUB_PERSONAL_ACCESS_TOKEN=<TOKEN_FROM_DEV_ENVIRONMENT>
go run cmd/github-mcp-server/main.go http --gh-host http://api.github.localhost --port 4567

@iancanderson
Add HTTP+SSE server command with graceful shutdown and logging middle…
aa9623c 
…ware

I'm not sure we want to merge this, but this allowed me to try dotcom chat action as an MCP host, using `mark3labs/mcp-go` in copilot-api, connecting to a localhost version of github-mcp-server via HTTP/SSE.

```
export GITHUB_PERSONAL_ACCESS_TOKEN=<TOKEN_FROM_DEV_ENVIRONMENT>
go run cmd/github-mcp-server/main.go http --gh-host http://api.github.localhost --port 4567
```
@israpasos
Copy link

HTTP+SSE support is crucial for deployments like Coolify, where services run in isolated containers and need network-based communication rather than relying on Docker socket mounts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juruen juruen Awaiting requested review from juruen juruen will be requested when the pull request is marked ready for review juruen is a code owner

@SamMorrowDrums SamMorrowDrums Awaiting requested review from SamMorrowDrums SamMorrowDrums will be requested when the pull request is marked ready for review SamMorrowDrums is a code owner

@williammartin williammartin Awaiting requested review from williammartin williammartin will be requested when the pull request is marked ready for review williammartin is a code owner

@toby toby Awaiting requested review from toby toby will be requested when the pull request is marked ready for review toby is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
server
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@iancanderson @israpasos @SamMorrowDrums