T05 · Unauthorized Access and Privilege Escalation
Error
- Location
- references/mcp-endpoints.md:5
- Finding
- Hard-Coded Unauthenticated MCP Trading Endpoint<![CDATA[ ## Vulnerability Details **File Location**: `references/mcp-endpoints.md:5-6` **Vulnerability Type**: Exposed unauthenticated trading control endpoint **Risk Level**: High ### Vulnerable Code ```text URL: `https://nnn8-antonbustrov.amvera.io/mcp/acc6ad15-3e20-4d20-9094-85bbb12e0780` Auth: none ``` The same document identifies broker write operations, including `Place_Order`, `Close_an_open_deal`, `Modify_Order`, and stop-loss/take-profit modification. ### Technical Analysis The Skill publishes a concrete MCP endpoint and explicitly states that authentication is not required. The endpoint is associated with both sensitive account-information operations and broker write operations. The workflow identifier embedded in the URL may function as a bearer-like capability. Even if the server implements controls not visible in this repository, publishing the complete endpoint unnecessarily expands the attack surface and violates least-privilege principles. Client-side instructions requiring user confirmation do not provide an authorization boundary: a caller interacting directly with the MCP service can attempt to bypass those instructions. The trading functionality legitimately requires access to a broker gateway, but it does not require distributing a specific unauthenticated deployment URL with the Skill. ### Attack Path 1. An attacker obtains the Skill package or reads the public reference document. 2. The attacker extracts the hard-coded MCP endpoint. 3. The attacker connects to the endpoint without presenting credentials. 4. The attacker enumerates or invokes exposed MCP tools. 5. If server-side authorization is absent as documented, the attacker reads account information or attempts broker write operations directly, without the Skill's user-confirmation workflow. ### Impact Assessment A successful exploit could expose account balances, margin information, open deals, and order history. If write operations are reachable without independent serve ...[truncated 269 chars]
- Remediation
- <![CDATA[ ## Remediation Suggestions 1. Remove the concrete deployment URL from the repository and replace it with a placeholder. 2. Rotate the exposed workflow identifier and invalidate the existing endpoint. 3. Require strong authentication, such as short-lived OAuth tokens or mutually authenticated TLS. 4. Enforce authorization on the MCP server for every operation, particularly broker write operations. 5. Separate read-only market-data tools from account and trading tools using distinct credentials and service identities. 6. Apply network allowlists, rate limiting, audit logging, and anomaly detection. 7. Require server-side transaction confirmation or approval for order placement, modification, cancellation, and closure. 8. Never rely on agent instructions such as “only with user confirmation” as the sole access-control mechanism. ]]>
