Skip to content

Commit fdf53d5

Browse files
committed
add back confused deputy
1 parent 664f942 commit fdf53d5

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

docs/specification/draft/basic/authorization.mdx

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -297,3 +297,15 @@ Authorization servers **MUST** take precautions to prevent redirecting user agen
297297

298298
Authorization servers **SHOULD** only automatically redirect the user agent if it trusts the redirection URI. If the URI is not trusted, the authorization server MAY inform the user and rely on the user to make the correct decision.
299299

300+
### 3.4 Confused Deputy Problem
301+
302+
An attacker can exploit OAuth proxy configurations that share third-party client credentials
303+
across multiple users.
304+
305+
When an MCP server fronts an authorization server that does not support dynamic client
306+
registration, the MCP server will use a static client ID to acquire credentials for the
307+
upstream API.
308+
309+
If the the backing authorization server sets cookies after user consent, an attacker can craft malicious authorization requests that bypass consent flows for previously authorized applications.
310+
311+
MCP servers using a static client_id for a backing service MUST require explicit approval for each newly registered dynamic client prior to forwarding requests to the backing authorization server for user consent.

0 commit comments

Comments
 (0)