Skip to content

Document ChatGPT Desktop configuration via ngrok tunnel#882

Merged
danbarr merged 2 commits into
mainfrom
add-chatgpt-desktop-ngrok-docs
May 19, 2026
Merged

Document ChatGPT Desktop configuration via ngrok tunnel#882
danbarr merged 2 commits into
mainfrom
add-chatgpt-desktop-ngrok-docs

Conversation

@danbarr
Copy link
Copy Markdown
Collaborator

@danbarr danbarr commented May 18, 2026

Description

Adds a ChatGPT Desktop section to the client compatibility guide. ChatGPT Desktop only accepts MCP servers over HTTPS (no command-based config like Claude Desktop), so the new section documents exposing a local ToolHive workload via thv proxy tunnel with ngrok, including a worked example and the connector setup steps.

The existing ChatGPT Desktop row in the compatibility table now links to this dedicated section instead of the generic STDIO-only configuration.

While here, also simplifies the existing STDIO-only workaround (for Claude Desktop):

  • Drops --transport sse from both example commands. SSE is deprecated, and the workarounds work with whatever transport the server runs by default.
  • Removes the stepwise "How it works" preambles in favor of "run an MCP server with ToolHive as usual, then apply the workaround."
  • Renames the second positional arg in the thv proxy tunnel example from osv to osv-tunnel so the two arguments are visibly distinct.

Type of change

  • Documentation update

Submitter checklist

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Copilot AI review requested due to automatic review settings May 18, 2026 17:18
@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs-website Ready Ready Preview, Comment May 19, 2026 3:25pm

Request Review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds dedicated documentation for connecting ChatGPT Desktop to local ToolHive-managed MCP servers by exposing them over HTTPS via thv proxy tunnel (ngrok), and updates the client compatibility table to link to this new section.

Changes:

  • Updates the ChatGPT Desktop row in the supported-clients table to link to a new dedicated section.
  • Adds a “ChatGPT Desktop configuration” section describing ngrok tunneling via thv proxy tunnel, including prerequisites, an example, and connector setup steps.

Comment thread docs/toolhive/reference/client-compatibility.mdx Outdated
@danbarr danbarr force-pushed the add-chatgpt-desktop-ngrok-docs branch from 30f9fc5 to 6a9ce78 Compare May 18, 2026 17:26
@danbarr danbarr marked this pull request as draft May 18, 2026 17:41
@danbarr danbarr force-pushed the add-chatgpt-desktop-ngrok-docs branch from 6a9ce78 to a914029 Compare May 19, 2026 14:13
@danbarr danbarr marked this pull request as ready for review May 19, 2026 14:13
Comment thread docs/toolhive/reference/client-compatibility.mdx
Comment thread docs/toolhive/reference/client-compatibility.mdx
Comment thread docs/toolhive/reference/client-compatibility.mdx
Add a warning admonition spelling out the three security concerns
raised in review: unauthenticated tunnel, credential exposure for
auth-backed servers, and shared-tunnel risk in workspace connectors.
Point readers to vMCP for shared or credentialed setups.
@danbarr
Copy link
Copy Markdown
Collaborator Author

danbarr commented May 19, 2026

Thanks for flagging those security concerns @amirejaz, I've addressed all three in a warning block and pointed readers towards vMCP for a proper multi-tenant and authenticated approach.

@danbarr danbarr merged commit 18e8fbe into main May 19, 2026
4 checks passed
@danbarr danbarr deleted the add-chatgpt-desktop-ngrok-docs branch May 19, 2026 15:28
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.

3 participants