Skip to content

fix: protect MCP config credentials - #143

Open
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:fix/protect-mcp-config-credentials
Open

fix: protect MCP config credentials#143
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:fix/protect-mcp-config-credentials

Conversation

@vycdev2

Copy link
Copy Markdown
Contributor

Summary

  • create the generated MCP config with owner-only permissions
  • tighten permissions on existing config files before writing authentication credentials
  • cover permissive pre-existing config files with a regression test

Verification

  • npm run build — passed
  • node --test tests/mcpHttp.test.mjs — passed (3 tests)
  • npm test — passed (195 tests)
  • git diff --check — passed

Risk

  • Low: the write path and JSON format are unchanged; only the file mode is tightened to 0600.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

Sign up for freeto 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.

1 participant

@vycdev2