[Bug]: DefaultRequestHandler runs follow-up messages on a task with the first request's contextvars
I maintainer di solito rispondono entro 2 giorni
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
What happened?
With DefaultRequestHandler (v2), a follow-up message on an existing task runs AgentExecutor.execute() with the contextvars of the request that created the task, not the request that sent the follow-up. A common case is a reply after the task went input-required.
Any per-request contextvar set by ASGI middleware or by the ServerCallContextBuilder comes through stale: auth tokens, tenant or caller identity, request-scoped tracing. ServerCallContext.state is correct per message. Only contextvars are wrong.
Reproduced on 1.1.2 and 1.2.2, for both SendMessage and SendStreamingMessage. 0.3.x is not affected because its handler started a new producer per message.
Cause
ActiveTask.start() creates the producer with asyncio.create_task(self._run_producer()) during the first request, so the producer copies that request's context once. Later requests call ActiveTask.enqueue_request(), which only puts the RequestContext on _request_queue. The producer then calls execute() from the first request's context. A task parked in input-required keeps its ActiveTask (#1296), so a reply hours later still sees the first request's values, which may by then be an expired token.
Reproduction
import asyncio, contextvars, uuid
import httpx
from a2a.helpers import new_task_from_user_message
from a2a.server.agent_execution import AgentExecutor
from a2a.server.request_handlers import DefaultRequestHandler
from a2a.server.routes import create_jsonrpc_routes
from a2a.server.tasks import InMemoryTaskStore, TaskUpdater
from a2a.types import AgentCapabilities, AgentCard
from starlette.applications import Starlette
REQUEST_TAG = contextvars.ContextVar("request_tag", default="unset")
class TagMiddleware:
def __init__(self, app):
self.app = app
async def __call__(self, scope, receive, send):
token = REQUEST_TAG.set(dict(scope.get("headers", [])).get(b"x-tag", b"unset").decode())
try:
await self.app(scope, receive, send)
finally:
REQUEST_TAG.reset(token)
class Executor(AgentExecutor):
async def execute(self, context, event_queue):
print(f"execute({context.get_user_input()!r}) sees request_tag={REQUEST_TAG.get()!r}")
task = context.current_task or new_task_from_user_message(context.message)
updater = TaskUpdater(event_queue, task.id, task.context_id)
if context.current_task:
await updater.complete()
else:
await event_queue.enqueue_event(task)
await updater.requires_input()
async def cancel(self, context, event_queue):
pass
card = AgentCard(name="a", description="a", version="1", capabilities=AgentCapabilities())
handler = DefaultRequestHandler(agent_executor=Executor(), task_store=InMemoryTaskStore(), agent_card=card)
app = Starlette(routes=create_jsonrpc_routes(request_handler=handler, rpc_url="/"))
app.add_middleware(TagMiddleware)
async def send(client, tag, text, task=None):
message = {"messageId": str(uuid.uuid4()), "role": "ROLE_USER", "parts": [{"text": text}]}
if task:
message |= {"taskId": task["id"], "contextId": task["contextId"]}
body = {"jsonrpc": "2.0", "id": 1, "method": "SendMessage", "params": {"message": message}}
resp = await client.post("/", json=body, headers={"x-tag": tag, "A2A-Version": "1.0"})
return resp.json()["result"]["task"]
async def main():
async with httpx.AsyncClient(transport=httpx.ASGITransport(app=app), base_url="http://t") as client:
task = await send(client, "request-1", "book a flight")
await send(client, "request-2", "Friday", task)
asyncio.run(main())
Actual:
execute('book a flight') sees request_tag='request-1'
execute('Friday') sees request_tag='request-1'
Expected:
execute('book a flight') sees request_tag='request-1'
execute('Friday') sees request_tag='request-2'
Proposed fix
Capture the sender's context in enqueue_request() and run execute() in it:
async def enqueue_request(self, request_context):
request_id = uuid.uuid4()
await self._request_queue.put((request_context, request_id, contextvars.copy_context()))
return request_id
# in _run_producer()
request_context, request_id, sender_context = await self._request_queue.get()
...
await sender_context.run(
asyncio.create_task,
self._agent_executor.execute(request_context, self._event_queue_agent),
)
I applied this patch to 1.2.2 locally and the reproduction prints request-2 for the follow-up, for both SendMessage and SendStreamingMessage. Context.run(asyncio.create_task, ...) works on Python 3.10. Awaiting the child task keeps cancellation working, because cancelling the producer cancels the child. The MCP Python SDK fixed the same problem this way in modelcontextprotocol/python-sdk#2298.
Downstream workaround
bedrock-agentcore works around this today without touching a2a internals. It wraps SimpleRequestContextBuilder to store contextvars.copy_context() in call_context.state, and wraps the executor to run execute() in that copy (aws/bedrock-agentcore-sdk-python#690). A fix here would let downstream projects delete that code, and it would also cover users who build DefaultRequestHandler directly.
Relevant log output
No response
Code of Conduct
- I agree to follow this project's Code of Conduct
- Lingua principale
- Python
- Stelle
- 2.2k
- Fork
- 509
- Merge medio
- 3g 11h
- PR unite (30g)
- 43
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi 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 a2aproject/a2a-python
-
[Bug]: REST task/request id sanitizationForse già presa @Linux2010 l’ha presa 101 giorni fa. Apertamaintainers-only
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
a2aproject/a2a-python#805 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
[Bug]: After a streamed task is cancelled, its background producer never finishesForse già presa @rohityan l’ha presa oggi. Aperta
a2aproject/a2a-python#1322 · 2 assegnatari ·
I maintainer di solito rispondono entro 2 giorni
-
v0.3 gRPC and REST SendMessage without configuration run non-blockingForse già presa @rohityan l’ha presa 1 giorno fa. Aperta
a2aproject/a2a-python#1321 · 2 assegnatari ·
I maintainer di solito rispondono entro 2 giorni
-
[Bug]: Push notification store failure rewrites a completed task as FAILED (DefaultRequestHandlerV2)Forse già presa @rohityan l’ha presa 2 giorni fa. Aperta
a2aproject/a2a-python#1313 · 2 assegnatari ·
I maintainer di solito rispondono entro 2 giorni
-
v0.3 JSON-RPC and REST GetTask return no history when history_length is 0Forse già presa @rohityan l’ha presa 3 giorni fa. Aperta
a2aproject/a2a-python#1311 · 2 assegnatari ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di a2aproject/a2a-python
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
Vector35/community-plugins#376 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
py-econometrics/pyfixest#1883 ·
I maintainer di solito rispondono entro 1 giorno
-
bad links in rfc5890.htmlAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
ietf-tools/rfc2html#81 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
mysql/mysql-operator#60 ·
-
Python: Bug: split_plaintext_paragraph / split_markdown_paragraph can return a chunk larger than max_tokensForse già presa @xThreeh l’ha presa oggi. Apertapython triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
microsoft/semantic-kernel#14566 ·
I maintainer di solito rispondono entro 4 giorni