mcp-documentation-server

Security Findings (11)

• HighExcessive Permission Surface ×3

  • search_documents:tool declares network permission
  • search_all_documents:tool declares network permission
  • get_ui_url:tool declares network permission

  Rule: AS-002Fix: Tool requests broad permissions (exec/fs/network). Validate input parameters using Enums where possible, and restrict file system operations to explicit allowed directories.
• MediumExcessive Permission Surface ×5

  • delete_document:tool declares fs permission
  • get_ui_url:tool declares fs permission
  • get_uploads_path:tool declares fs permission
  • process_uploads:tool declares fs permission
  • list_uploads_files:tool declares fs permission

  Rule: AS-002Fix: Tool requests broad permissions (exec/fs/network). Validate input parameters using Enums where possible, and restrict file system operations to explicit allowed directories.
• LowDoS Resilience — Missing Rate Limit / Timeout ×3

  • search_documents:tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration
  • search_all_documents:tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration
  • get_ui_url:tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration

  Rule: AS-011Fix: Declare explicit rate-limit, timeout, and retry configuration for all network and execution tools. Implement exponential back-off and surface resource state to the calling agent.