-
Notifications
You must be signed in to change notification settings - Fork 45
Change default port from 5000 to 8000 #417
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
1899899
47c1367
4bd07ba
373b34d
93e607b
bf4cba2
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 |
---|---|---|
@@ -0,0 +1,20 @@ | ||
### Change default port from 5000 to 8000 - @DaleSeo PR #417 | ||
|
||
The default server port has been changed from `5000` to `8000` to avoid conflicts with common development tools and services that typically use port 5000 (such as macOS AirPlay, Flask development servers, and other local services). | ||
|
||
**Migration**: If you were relying on the default port 5000, you can continue using it by explicitly setting the port in your configuration file or command line arguments. | ||
|
||
- Before | ||
|
||
```yaml | ||
transport: | ||
type: streamable_http | ||
``` | ||
- After | ||
```yaml | ||
transport: | ||
type: streamable_http | ||
port: 5000 | ||
``` |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -94,7 +94,7 @@ impl Transport { | |
} | ||
|
||
fn default_port() -> u16 { | ||
5000 | ||
8000 | ||
} | ||
|
||
fn default_stateful_mode() -> bool { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -69,16 +69,16 @@ There are operations located at `./operations/*.graphql` for you to use in your | |
docker run \ | ||
-it --rm \ | ||
--name apollo-mcp-server \ | ||
-p 5000:5000 \ | ||
-p 8000:8000 \ | ||
-v $PWD/graphql/TheSpaceDevs:/data \ | ||
ghcr.io/apollographql/apollo-mcp-server:latest \ | ||
--http-port 5000 \ | ||
--http-port 8000 \ | ||
--schema api.graphql \ | ||
--operations operations \ | ||
--endpoint https://thespacedevs-production.up.railway.app/ | ||
|
||
``` | ||
|
||
2. Add the MCP SSE port to your MCP Server configuration for the client appliction you are running. If you are running locally, the server link will be `http://127.0.0.1:5000/sse`. | ||
2. Add the MCP SSE port to your MCP Server configuration for the client appliction you are running. If you are running locally, the server link will be `http://127.0.0.1:8000/sse`. | ||
|
||
|
||
_Note: Claude Desktop currently doesn't support SSE_ | ||
|
||
|
@@ -87,7 +87,7 @@ _Note: Claude Desktop currently doesn't support SSE_ | |
"mcpServers": { | ||
"thespacedevs": { | ||
"type": "sse", | ||
"url": "http://127.0.0.1:5000/sse>" | ||
"url": "http://127.0.0.1:8000/sse>" | ||
} | ||
} | ||
} | ||
|
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.
Could we just remove this section? I assume this troubleshooting step was only added because of the port use region that incentivized us to change the default.
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.
This is probably still relevant. I often forgot to kill the server before starting a new one. 😂