Skip to content

Add network transport section #63

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

Merged
merged 3 commits into from
Jul 21, 2025
Merged

Add network transport section #63

merged 3 commits into from
Jul 21, 2025

Conversation

danbarr
Copy link
Collaborator

@danbarr danbarr commented Jul 17, 2025

Description

Added a network transport section to the running MCP servers guide, to explain how network transports and the --target-port flag interact with the host/container ports and the proxy.

Direct link to the new section in the preview: https://docs-website-git-add-net-transport-info-stacklok.vercel.app/toolhive/guides-cli/run-mcp-servers#configure-network-transport

Related issues/PRs

N/A

Screenshots

Included diagrams to visualize the default behavior, and using --target-port:
image
image

Merge checklist

Content

  • (N/A) New pages include a frontmatter section with title and description at a minimum
  • (N/A) Sidebar navigation (sidebars.ts) updated for added, deleted, reordered, or renamed files
  • (N/A) Redirects added to vercel.json for moved, renamed, or deleted pages (i.e., if the URL slug changed)

Reviews

  • Content has been reviewed for technical accuracy (@aponcedeleonch, you've been doing a lot with transports lately so I'd appreciate your review!)
  • Content has been reviewed for spelling, grammar, and style

Copy link

vercel bot commented Jul 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 21, 2025 0:30am

@danbarr danbarr requested a review from aponcedeleonch July 17, 2025 22:54
aponcedeleonch
aponcedeleonch previously approved these changes Jul 18, 2025
Copy link
Contributor

@aponcedeleonch aponcedeleonch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some really minor comments, but it looks good. I always like a nice looking diagram

Copy link
Contributor

@aponcedeleonch aponcedeleonch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@danbarr danbarr merged commit 6fe96e9 into main Jul 21, 2025
5 checks passed
@danbarr danbarr deleted the add-net-transport-info branch July 21, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants