Skip to content

add MCP Gateway #22935

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

ArthurFlag
Copy link
Contributor

Description

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

Copy link

netlify bot commented Jun 27, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 2a48c28
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/686293f54e10400008a199b3
😎 Deploy Preview https://deploy-preview-22935--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added hugo Updates related to hugo area/ai labels Jun 27, 2025
@ArthurFlag ArthurFlag force-pushed the ENGDOCS-2781-mcp-gateway branch 2 times, most recently from d82f725 to 5b8aeb3 Compare June 27, 2025 10:22
@ArthurFlag ArthurFlag force-pushed the ENGDOCS-2781-mcp-gateway branch from 5b8aeb3 to 9e33ba7 Compare June 27, 2025 10:25
Copy link

@nunocoracao nunocoracao left a comment

Choose a reason for hiding this comment

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

@ArthurFlag left some comments
@dgageot could you have a look at my comments - I think devils in the details on this one

@craig-osterhout craig-osterhout mentioned this pull request Jun 27, 2025
2 tasks
@ArthurFlag ArthurFlag force-pushed the ENGDOCS-2781-mcp-gateway branch from 696fe54 to 2a48c28 Compare June 30, 2025 13:41
@ArthurFlag ArthurFlag marked this pull request as ready for review July 2, 2025 08:16
@craig-osterhout
Copy link
Contributor

The left nav auto-generates a dropdown if using a folder, even with only a single topic in it. Suggest remove the folder (mcp-gateway), rename the index.md to the folder name (mcp-gateway.md), and update any relative links. I think everything should generate the same except the dropdown.

Copy link
Contributor

@aevesdocker aevesdocker left a comment

Choose a reason for hiding this comment

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

Picky style guide nits, rest looks great

By unifying multiple MCP servers into a single, secure endpoint, the MCP Gateway offers
the following benefits:

- Secure by Default: MCP servers run in isolated Docker containers with restricted
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Secure by Default: MCP servers run in isolated Docker containers with restricted
- Secure by default: MCP servers run in isolated Docker containers with restricted


- Secure by Default: MCP servers run in isolated Docker containers with restricted
privileges, network access, and resource usage.
- Unified Management: One gateway endpoint centralizes configuration, credentials,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Unified Management: One gateway endpoint centralizes configuration, credentials,
- Unified management: One gateway endpoint centralizes configuration, credentials,

privileges, network access, and resource usage.
- Unified Management: One gateway endpoint centralizes configuration, credentials,
and access control for all MCP servers.
- Enterprise Observability: Built-in monitoring, logging, and filtering tools ensure
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Enterprise Observability: Built-in monitoring, logging, and filtering tools ensure
- Enterprise observability: Built-in monitoring, logging, and filtering tools ensure


- Developers: Deploy MCP servers locally and in production using Docker Compose,
with built-in support for protocol handling, credential management, and security policies.
- Security Teams: Achieve enterprise-grade isolation and visibility into AI tool
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Security Teams: Achieve enterprise-grade isolation and visibility into AI tool
- Security teams: Achieve enterprise-grade isolation and visibility into AI tool


## Key features

- Server Management: List, inspect, and call MCP tools, resoures and prompts from multiple servers
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Server Management: List, inspect, and call MCP tools, resoures and prompts from multiple servers
- Server management: List, inspect, and call MCP tools, resources and prompts from multiple servers


- Server Management: List, inspect, and call MCP tools, resoures and prompts from multiple servers
- Container-based Servers: Run MCP servers as Docker containers with proper isolation
- Secrets Management: Secure handling of API keys and credentials via Docker Desktop
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Secrets Management: Secure handling of API keys and credentials via Docker Desktop
- Secrets management: Secure handling of API keys and credentials via Docker Desktop

- Server Management: List, inspect, and call MCP tools, resoures and prompts from multiple servers
- Container-based Servers: Run MCP servers as Docker containers with proper isolation
- Secrets Management: Secure handling of API keys and credentials via Docker Desktop
- Server Catalog: Manage and configure multiple MCP catalogs
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Server Catalog: Manage and configure multiple MCP catalogs
- Server catalog: Manage and configure multiple MCP catalogs

- Container-based Servers: Run MCP servers as Docker containers with proper isolation
- Secrets Management: Secure handling of API keys and credentials via Docker Desktop
- Server Catalog: Manage and configure multiple MCP catalogs
- Dynamic Discovery and Reloading: Automatic tool, prompt, and resource discovery from running servers
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Dynamic Discovery and Reloading: Automatic tool, prompt, and resource discovery from running servers
- Dynamic discovery and reloading: Automatic tool, prompt, and resource discovery from running servers


> [!IMPORTANT]
> These instructions are mostly suitable for testing purposes.
> We otherwise recommend building from the sources. See the [readme](https://github.com/docker/mcp-gateway?tab=readme-ov-file#installation).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
> We otherwise recommend building from the sources. See the [readme](https://github.com/docker/mcp-gateway?tab=readme-ov-file#installation).
> We otherwise recommend building from the sources. See the [README](https://github.com/docker/mcp-gateway?tab=readme-ov-file#installation).

| macOS | `docker-mcp` | `$HOME/.docker/cli-plugins` |
| Windows | `docker-mcp.exe` | `%USERPROFILE%\.docker\cli-plugins` |

Or copy it into one of these folders for installing it system-wide.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Or copy it into one of these folders for installing it system-wide.
Or copy it into one of these folders for installing it system-wide:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants