Skip to content

Commit bbcab63

Browse files
committed
fix numbering
1 parent 330723a commit bbcab63

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

docs/specification/draft/basic/authorization.mdx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ as outlined in [RFC 9700](https://datatracker.ietf.org/doc/html/rfc9700).
266266

267267
MCP authorization servers SHOULD enforce token expiration and rotation to limit the window of exploitation.
268268

269-
### 3.3 Token Interception
269+
### 3.2 Token Interception
270270
An attacker positioned between MCP clients and MCP servers can intercept tokens via [Man-in-the-Middle (MITM)](https://en.wikipedia.org/wiki/Man-in-the-middle_attack) attacks.
271271

272272
To mitigate the risk of this threat:
@@ -275,7 +275,7 @@ To mitigate the risk of this threat:
275275
1. MCP clients **MUST** implement PKCE according to [OAuth 2.1 section 7.5.2](https://www.ietf.org/archive/id/draft-ietf-oauth-v2-1-12.html#name-countermeasures). PKCE helps prevent authorization code interception attacks by requiring clients to create a secret verifier-challenge pair, ensuring that only the original requestor can exchange an authorization code for tokens.
276276
1. All redirect URIs **MUST** be either `localhost` or use HTTPS to prevent token and code interception.
277277

278-
### 3.4 Open Redirection
278+
### 3.3 Open Redirection
279279
An attacker may craft malicious redirect URIs to direct users to phishing sites
280280
and intercept credentials during the authorization flow.
281281

@@ -287,7 +287,7 @@ MCP clients **SHOULD** use and verify state parameters in the authorization code
287287
and discard any results that do not include or have a mis-match with the original state.
288288

289289

290-
### 3.5 Confused Deputy Problem
290+
### 3.4 Confused Deputy Problem
291291

292292
An attacker can exploit OAuth proxy configurations that share third-party client credentials
293293
across multiple users.

0 commit comments

Comments
 (0)