pierrejeambrun commented on code in PR #58344:
URL: https://github.com/apache/airflow/pull/58344#discussion_r2557086297
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/xcom.py:
##########
@@ -80,6 +80,7 @@ def get_xcom_entry(
stringify: Annotated[bool, Query()] = False,
) -> XComResponseNative | XComResponseString:
"""Get an XCom entry."""
+ # FastAPI decodes path params, use the value directly.
Review Comment:
```suggestion
```
##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/xcom.py:
##########
@@ -314,7 +315,7 @@ def update_xcom_entry(
) -> XComResponseNative:
"""Update an existing XCom entry."""
# Check if XCom entry exists
- xcom_new_value = XComModel.serialize_value(patch_body.value)
+ # FastAPI decodes path params, use the value directly.
Review Comment:
Same etc.. to remove
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
