-
Notifications
You must be signed in to change notification settings - Fork 16.6k
fix: xcom_push raises 422 error when value is set to None #55080
fix: xcom_push raises 422 error when value is set to None #55080kaxil merged 3 commits intoapache:mainfrom
Conversation
jscheffl
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting find! I am sure we had the same error in 2.9 or 2.10 when I added the JSON view to XCom that I was running into the same trap and forgot to check "null" values. Thanks for fixing!
Can you fix the small nit? Then good to merge I think
airflow-core/src/airflow/api_fastapi/execution_api/versions/__init__.py
Outdated
Show resolved
Hide resolved
airflow-core/src/airflow/api_fastapi/execution_api/versions/v2025_09_23.py
Outdated
Show resolved
Hide resolved
|
I propose we should backport this to 3.0-line as well. |
|
Yeah, it is a bug-fix |
|
@jscheffl the backport tool won't work post merge. Manual port will be required |
|
@rawwar could you please create a manual backport? |
|
@amoghrajesh did it already in #55095 |
|
I have a question regarding this fix, I am on airflow version 3.0.6 and using EDIT: I see the fix in the changelog here in v3.1.0. Do I have to update to 3.1.0? |
|
@prakharcode Yes, we do not backport bugfixes to older versions, it was fixed and merged for 3.1.0, you need to upgrade to benefit from this. (or cherry pick that patch and build your own airflow, but that's more effort) |
|
Also seeing this on Airflow 3.0.6. AWS MWAA currently does not offer Airflow 3.1 Environments, so it's very unfortunate this bug fix cannot be applied for older versions. |
It can be. Those who run managed service can backport any change to their versions. This is open source and people work here mostly as volunteers and you get what you pay for - i.e. you pay 0 to airflow community and you pay maintenance fee to whoever you bought managed service frome. It seems pretty eggregious to demand people who work for free to do something to do "extra effort" rather than those who get money for maintenace. I guess you should be demanding more from those who your money are directed to, rather than those who you paid exactly 0. @emilsauer . Or am I missing something? |
I appreciate the context. I think there was a misunderstanding of my intent. I certainly wasn't demanding a backport or suggesting extra effort from the community. My comment was meant to document that the issue persists in 3.0.6 for the benefit of other users who, like me, are currently restricted to that version. Since this bug is not trivial to diagnose, I wanted to provide a reference point for others to share findings or workarounds. I just wanted to clarify my intent here. Thanks for the work on the project. |