This is an automated email from the ASF dual-hosted git repository.

pierrejeambrun pushed a commit to branch v3-3-test
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/v3-3-test by this push:
     new 439704157a3 [v3-3-test] Drop the stale RFC 9457 TODO on the 
task-instance-run endpoint (#71552) (#71873)
439704157a3 is described below

commit 439704157a3e143b9b84bd4361d128f97a37fa7c
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Aug 20 12:31:32 2026 +0200

    [v3-3-test] Drop the stale RFC 9457 TODO on the task-instance-run endpoint 
(#71552) (#71873)
    
    The whole execution API returns errors as ``{"reason": ..., "message":
    ..., ...extras}`` — that shape is the de-facto contract now (parsed by
    the task SDK client at ``sdk/api/client.py``) and is applied uniformly
    across every route in the module. This TODO predates that convention
    and calling it out only on one 409 site is misleading: any single-
    endpoint migration to RFC 9457 would break the SDK parser and leave
    every other endpoint on the existing shape.
    
    Removing the comment so it stops attracting drive-by "fix RFC 9457"
    PRs; a real migration would need to be a cross-cutting change of the
    whole execution API plus the SDK parsers.
    (cherry picked from commit b2cae5d9950ae2962b4757df49e741ba96da1e14)
    
    
    related: #69360
    
    Co-authored-by: Pierre Jeambrun <[email protected]>
---
 .../src/airflow/api_fastapi/execution_api/routes/task_instances.py   | 5 -----
 1 file changed, 5 deletions(-)

diff --git 
a/airflow-core/src/airflow/api_fastapi/execution_api/routes/task_instances.py 
b/airflow-core/src/airflow/api_fastapi/execution_api/routes/task_instances.py
index 41ecf49b053..c5415e45e77 100644
--- 
a/airflow-core/src/airflow/api_fastapi/execution_api/routes/task_instances.py
+++ 
b/airflow-core/src/airflow/api_fastapi/execution_api/routes/task_instances.py
@@ -214,11 +214,6 @@ def ti_run(
             previous_state=previous_state,
         )
 
-        # TODO: Pass a RFC 9457 compliant error message in "detail" field
-        # https://datatracker.ietf.org/doc/html/rfc9457
-        # to provide more information about the error
-        # FastAPI will automatically convert this to a JSON response
-        # This might be added in FastAPI in 
https://github.com/fastapi/fastapi/issues/10370
         raise HTTPException(
             status_code=status.HTTP_409_CONFLICT,
             detail={

Reply via email to