MCP tools/list exposes tools the caller cannot invoke (security not evaluated)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
Direzione di ricerca
Inizia con ApiPlatform\Mcp\Server\ListHandler e il punto di ingresso tools/list, quindi confrontalo con il comportamento di sicurezza di tools/call descritto in #8435. Esamina le domande sulla policy indicate per la sicurezza a livello di operazione e le espressioni con object, e usa i passaggi di riproduzione per verificare che gli strumenti negati non espongano più metadati in nessuna delle due modalità di listener.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
ApiPlatform\Mcp\Server\ListHandler enumerates every MCP tool when answering tools/list, regardless of the security expression declared on #[McpTool]. A caller who is not allowed to invoke a tool can still discover it: its name, description and full input schema are returned.
This is an information-disclosure issue rather than an access-control bypass — invocation itself is correctly denied. But the tool name, its description and the shape of its arguments frequently reveal internal domain structure that the security expression was meant to keep hidden.
Found while reviewing #8435, which fixed the related (and more severe) problem that security was a complete no-op on the tools/call path when use_symfony_listeners: true. That fix closed invocation; listing was out of its scope.
Current behaviour
tools/call—security,securityPostDenormalize,securityPostValidationand Link-levelsecurityare all enforced (as of #8435, on bothuse_symfony_listenersmodes).tools/list— no security expression is evaluated. Every registered tool is listed for every caller, anonymous included.
Expected behaviour
tools/list should omit tools whose security expression evaluates to false for the current caller, mirroring how collection operations filter what a user may see.
Open design questions worth settling before implementation:
- Only the operation-level
securityexpression can reasonably be evaluated at list time —securityPostDenormalizeandsecurityPostValidationneed an object and arguments that do not exist yet. The listing filter should therefore be documented as best-effort onsecurityonly. - Expressions referencing
objectcannot be evaluated during listing. Those tools need a defined policy: listed, hidden, or a configuration flag.
Reproduce
- Declare a tool with a
securityexpression that denies anonymous callers:
#[McpTool(name: 'secured_tool', security: "is_granted('ROLE_ADMIN')")]
- Issue an unauthenticated
tools/listJSON-RPC call to/mcp. secured_toolappears in the response, with its description and input schema.- A subsequent
tools/callfor it is correctly rejected.
Reproduces in both use_symfony_listeners: true and the default mode — the listing path never consulted security in either.
Notes
- Pre-existing; not a regression from #8435.
ApiPlatform\Mcp\Server\Handleris marked@experimental, so a behaviour change here is acceptable.
- Lingua principale
- PHP
- Stelle
- 2.6k
- Fork
- 982
- Merge medio
- 1g 16h
- PR unite (30g)
- 59
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di api-platform/core
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
api-platform/core#8573 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
api-platform/core#8571 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
api-platform/core#8564 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
api-platform/core#8495 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
api-platform/core#8471 ·
Tutte le issue di api-platform/core
Issue simili
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
Automattic/safe-publish#594 ·
-
HttpClient
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
symfony/symfony-docs#23092 ·
-
sync-en
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
-
sync-en
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
-
Перевод устарел
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100