Skip to content

add --allow-http flag - #28

Merged
geelen merged 1 commit into
punkpeye:mainfrom
kjosk:add_allow-http_flag
Apr 16, 2025
Merged

geelen merged 1 commit into
punkpeye:mainfrom
kjosk:add_allow-http_flag

Conversation

@kjosk

@kjosk kjosk commented Apr 16, 2025

Copy link
Copy Markdown
Contributor

Add --allow-http flag for private network usage

Overview

This PR adds a --allow-http flag to allow HTTP connections in trusted private networks such as AWS VPC. Currently, the client enforces HTTPS except for localhost connections, but there are valid use cases where HTTPS is unnecessary or impractical within secure private networks.

Changes

  • Added --allow-http flag to bypass HTTPS requirement
  • Updated error message to be more descriptive about when HTTP is allowed
  • Maintained existing localhost exception while adding the new flag option

Use Case

In private network environments like AWS VPC, where network traffic is already secured by infrastructure-level controls, requiring HTTPS can add unnecessary complexity. Some specific scenarios include:

  • Internal services running within VPC that communicate only through private networks
  • Development/testing environments in isolated networks

Security Considerations

The --allow-http flag should only be used in environments where:

  • Network traffic is contained within a secure private network (e.g., AWS VPC)
  • Infrastructure-level security controls are in place
  • Traffic cannot be intercepted by external parties

Example Usage

      "args": [
        "mcp-remote",
        "http://internal-service.vpc/sse",
        "--allow-http"
      ]

@geelen
geelen merged commit a206450 into punkpeye:main Apr 16, 2025
@geelen

geelen commented Apr 16, 2025

Copy link
Copy Markdown
Collaborator

Thanks for this! Released v0.0.21 with this included.

@kjosk

kjosk commented Apr 16, 2025

Copy link
Copy Markdown
Contributor Author

Thanks for merging the PR! Appreciate the quick review.

@coderabbitai coderabbitai Bot mentioned this pull request Aug 4, 2026
4 of 17 tasks
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