Skip to content

Conversation

@roomote
Copy link
Contributor

@roomote roomote bot commented Aug 8, 2025

This PR adds comprehensive documentation for integrating the BurpSuite MCP server with Roo-Code.

Changes

  • Created detailed documentation for BurpSuite MCP server setup and configuration
  • Added installation, configuration, and troubleshooting guides
  • Created MCP servers documentation directory structure (docs/mcp-servers/)
  • Updated main README with link to MCP server integrations

Related Issue

Fixes #6844

Documentation Added

  • docs/mcp-servers/burpsuite.md - Complete guide for BurpSuite MCP server integration
  • docs/mcp-servers/README.md - Index page for MCP server documentation

This documentation enables users to integrate BurpSuite with Roo-Code through the Model Context Protocol, allowing AI assistants to interact with Burp Suite for security testing and web application analysis.


Important

Adds documentation for BurpSuite MCP server integration with Roo-Code, including setup and troubleshooting guides.

  • Documentation:
    • Adds docs/mcp-servers/burpsuite.md for BurpSuite MCP server integration.
    • Creates docs/mcp-servers/README.md as an index for MCP server documentation.
    • Updates README.md with a link to MCP server integrations.
  • Structure:
    • Establishes docs/mcp-servers/ directory for MCP server documentation.
  • Guides:
    • Includes installation, configuration, and troubleshooting guides for BurpSuite MCP server.

This description was created by Ellipsis for a1b0374. You can customize this summary. It will automatically update as commits are pushed.

- Created comprehensive documentation for BurpSuite MCP server setup
- Added installation, configuration, and troubleshooting guides
- Created MCP servers documentation directory structure
- Updated main README with link to MCP server integrations

Fixes #6844
@roomote roomote bot requested review from cte, jr and mrubens as code owners August 8, 2025 12:04
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. documentation Improvements or additions to documentation labels Aug 8, 2025
Copy link
Contributor Author

@roomote roomote bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewing my own documentation is like proofreading a ransom note I wrote to myself.

"mcpServers": {
"burpsuite": {
"command": "java",
"args": ["-jar", "/path/to/mcp-server/build/libs/mcp-proxy-all.jar", "--sse-url", "http://127.0.0.1:9876"],
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Critical issue: The JAR file name here doesn't match what's created in the build step. The build creates burp-mcp-all.jar (line 44), but this references mcp-proxy-all.jar. This mismatch will cause the configuration to fail.

"mcpServers": {
"burpsuite": {
"command": "java",
"args": ["-jar", "/path/to/mcp-proxy-all.jar", "--sse-url", "http://127.0.0.1:9876"],
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same JAR file name issue here - should be burp-mcp-all.jar to match the build output:

2. Build the extension JAR file:

```bash
./gradlew embedProxyJar
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This command won't work on Windows. Could we add a note for Windows users?

bash
# On Linux/macOS:
./gradlew embedProxyJar

# On Windows:
gradlew.bat embedProxyJar

### Permission Errors

- Ensure the JAR file path is correct and accessible
- On macOS/Linux, ensure the JAR file has execute permissions: `chmod +x mcp-proxy-all.jar`
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this necessary? JAR files don't need execute permissions since they're run through the Java interpreter. This command won't cause issues but might confuse users. Consider removing this line or clarifying that it's not typically needed.

Before setting up the BurpSuite MCP server, ensure you have:

1. **Burp Suite Professional or Community Edition** installed and running
2. **Java** installed and available in your system's PATH
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we specify which Java version is required? The BurpSuite MCP server might have specific version requirements (e.g., Java 11 or higher).


Before setting up the BurpSuite MCP server, ensure you have:

1. **Burp Suite Professional or Community Edition** installed and running
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be helpful to specify which versions of Burp Suite are compatible with this MCP server? This could prevent users from encountering compatibility issues.

@roomote roomote bot mentioned this pull request Aug 8, 2025
1 task
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Aug 8, 2025
@daniel-lxs
Copy link
Member

MCP Marketplace addition request

@daniel-lxs daniel-lxs closed this Aug 11, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Aug 11, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Add Burpsuite MCP Server

4 participants