add hostname to listen method #25
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added UI components from the copilot-mcp build with the following changes: - Created new ServerViewProvider class with event-based UI updates - Updated ServerCard component to support SSE servers - Updated types to support multiple server types - Added migrateServerConfigurations command - Extended package.json with new server type configuration 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
Added a detailed summary of our debugging process with incremental changes. Document identifies that the proposed API issue wasn't triggered by any of our changes, suggesting it must be related to something else in the problematic build. 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
Added more complete UI components from the problematic build: - Added badge.tsx, dropdown-menu.tsx, and separator.tsx components - Completely rewrote ServerCard.tsx with the enhanced UI from the non-working build - Fixed type errors in event handlers - Added dependencies for @radix-ui components The UI now correctly matches the more sophisticated version from the problematic build but maintains the working architecture previously established. 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
1. Fixed environment variable handling for servers, especially for FireCrawl API key - Added special handling to export API keys directly in shell commands - Improved environment variable merging and string handling - Added better error logging for environment variables 2. Enhanced UI state management - Fixed server enable/disable state tracking with proper persistence - Added immediate UI refresh after state changes - Fixed tool count display on server cards - Improved state synchronization between backend and frontend 3. Added GitHub workflows and updated documentation - Added release and version update workflows - Updated README and CHANGELOG with latest features - Ensured proper TypeScript variable naming to avoid conflicts
- Implemented dynamic tools extension generation for MCP servers - Removed dependency on @automatalabs/mcp-client-manager - Updated package.json and package-lock.json to use latest @modelcontextprotocol/sdk - Added find-cache-dir for better temporary file management - Refactored server configuration to remove server ID requirement - Enhanced tool and server registration process - Improved logging and error handling - Updated VSCode configuration and extension activation logic
- Implemented server name validation in App.tsx and ServerCard.tsx - Added regex validation to ensure server names only contain letters, numbers, dashes, and underscores - Improved form error handling for server name input - Prevents invalid server names from being submitted
feat: Dynamic MCP server tools extension with improved configuration - Implemented dynamic tools extension generation for MCP servers - Removed dependency on @automatalabs/mcp-client-manager - Updated package.json and package-lock.json to use latest @modelcontextprotocol/sdk - Added find-cache-dir for better temporary file management - Refactored server configuration to remove server ID requirement - Enhanced tool and server registration process - Improved logging and error handling - Updated VSCode configuration and extension activation logic - Implemented dynamic tools extension generation for MCP servers - Removed dependency on @automatalabs/mcp-client-manager - Updated package.json and package-lock.json to use latest @modelcontextprotocol/sdk - Added find-cache-dir for better temporary file management - Refactored server configuration to remove server ID requirement - Enhanced tool and server registration process - Improved logging and error handling - Updated VSCode configuration and extension activation logic
- Added command escaping for process servers to handle special characters - Updated tools extension to include additional server capabilities - Improved error logging in server connection process - Bumped extension version to 0.0.15
This commit removes the DEBUG_FINDINGS.md file.
Removed the 'args' array property from the server configuration schema, simplifying the configuration structure.
Update links in README.md
* refactor: refactored app * chore: remove esbuild.js file and related build logic * chore: rename files * chore: rev version
…8 - update version to 0.0.29 (VikashLoomba#19)
…ools image; clean up ChatHandler logging
DraganVeljovic
approved these changes
Mar 25, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Modified the Express server configuration to explicitly bind to the specified hostname instead of listening on all network interfaces (0.0.0.0). This change improves security by limiting server access to the intended hostname only (defaults to 'localhost').
Changes
Updated
app.listen()to use both port and hostname parametersDefault hostname remains 'localhost' for local development
Prevents unintended external access to the SSE server
Before
Server would listen on all network interfaces (0.0.0.0)
After
Server only listens on the specified hostname (defaults to localhost)