-
Notifications
You must be signed in to change notification settings - Fork 93
chore: add streamable http disclaimer #390
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
Conversation
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.
Pull Request Overview
This PR adds a security disclaimer and best practices guidance for the HTTP transport feature in the MongoDB MCP Server. The change aims to warn users about security considerations when using the streamable HTTP transport option.
- Adds a prominent security warning about HTTP transport risks
- Provides specific security measure recommendations for production deployments
- References official documentation for additional security considerations
@@ -230,6 +230,18 @@ With Atlas API credentials: | |||
|
|||
#### Option 6: Running as an HTTP Server | |||
|
|||
> **⚠️ Security Notice:** This server now supports Streamable HTTP transport for remote connections. **HTTP transport is NOT recommended for production use without implementing proper authentication and security measures.** |
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.
[nitpick] The term 'Streamable HTTP transport' should be consistently capitalized. Consider using 'streamable HTTP transport' (lowercase 's') to match standard technical writing conventions unless this is a proper noun or brand name.
> **⚠️ Security Notice:** This server now supports Streamable HTTP transport for remote connections. **HTTP transport is NOT recommended for production use without implementing proper authentication and security measures.** | |
> **⚠️ Security Notice:** This server now supports streamable HTTP transport for remote connections. **HTTP transport is NOT recommended for production use without implementing proper authentication and security measures.** |
Copilot uses AI. Check for mistakes.
Pull Request Test Coverage Report for Build 16449772986Details
💛 - Coveralls |
Proposed changes
Checklist