Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix: xcom_push raises 422 error when value is set to None #55080

Merged
kaxil merged 3 commits intoapache:mainfrom
rawwar:kalyan/xcom/none_value
Aug 30, 2025
Merged

fix: xcom_push raises 422 error when value is set to None #55080
kaxil merged 3 commits intoapache:mainfrom
rawwar:kalyan/xcom/none_value

Conversation

Copy link
Contributor

rawwar commented Aug 29, 2025

fixes #55062

rawwar requested review from amoghrajesh, ashb, bbovenzi, jscheffl, kaxil, pierrejeambrun, ryanahamilton and shubhamraj-git as code owners August 29, 2025 17:32
boring-cyborg bot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Aug 29, 2025
Copy link
Contributor Author

rawwar commented Aug 29, 2025

When XCom value was null in DB, UI showed error:

With the changes in the PR, it now shows "null"

rawwar changed the title allow xcom value to be None fix: xcom_push raises 422 error when value is set to None Aug 29, 2025
jscheffl approved these changes Aug 29, 2025
Copy link
Contributor

jscheffl left a 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

rawwar reacted with thumbs up emoji
kaxil reviewed Aug 29, 2025
kaxil reviewed Aug 29, 2025
kaxil approved these changes Aug 29, 2025
amoghrajesh approved these changes Aug 30, 2025
potiuk approved these changes Aug 30, 2025
kaxil merged commit b4cf3af into apache:main Aug 30, 2025
107 checks passed
jscheffl added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Aug 30, 2025
Copy link
Contributor

jscheffl commented Aug 30, 2025

I propose we should backport this to 3.0-line as well.

rawwar reacted with thumbs up emoji

jscheffl pushed a commit to jscheffl/airflow that referenced this pull request Aug 30, 2025
potiuk pushed a commit that referenced this pull request Aug 30, 2025
fixes #55062

(cherry picked from commit b4cf3af)

Co-authored-by: Kalyan R
mangal-vairalkar pushed a commit to mangal-vairalkar/airflow that referenced this pull request Aug 30, 2025
Copy link
Contributor

amoghrajesh commented Sep 2, 2025

Yeah, it is a bug-fix

Copy link
Contributor

amoghrajesh commented Sep 2, 2025

@jscheffl the backport tool won't work post merge. Manual port will be required

jscheffl reacted with thumbs up emoji

Copy link
Contributor

amoghrajesh commented Sep 2, 2025

@rawwar could you please create a manual backport?

Copy link
Contributor

jscheffl commented Sep 2, 2025

@amoghrajesh did it already in #55095

nothingmin pushed a commit to nothingmin/airflow that referenced this pull request Sep 2, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Sep 30, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 1, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 2, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 3, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 4, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 5, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 5, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 7, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 8, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 9, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 10, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 11, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 12, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 14, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 15, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 17, 2025
abdulrahman305 bot pushed a commit to qenex-ai/airflow that referenced this pull request Oct 19, 2025
Copy link
Contributor

prakharcode commented Jan 28, 2026 *
edited
Loading

I have a question regarding this fix, I am on airflow version 3.0.6 and using apache/airflow:slim-3.0.6-python3.11. I don't see this fix in that image. Is this not available in 3.0.6 or am I missing something?

EDIT: I see the fix in the changelog here in v3.1.0. Do I have to update to 3.1.0?

Copy link
Member

pierrejeambrun commented Jan 28, 2026

@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)

Copy link

emilsauer commented Feb 3, 2026

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.

meldiner pushed a commit to faros-ai/airflow that referenced this pull request Feb 6, 2026
Copy link
Member

potiuk commented Feb 15, 2026

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?

Copy link

emilsauer commented Feb 16, 2026

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.

pierrejeambrun and prakharcode reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

potiuk potiuk approved these changes

kaxil kaxil approved these changes

amoghrajesh amoghrajesh approved these changes

jscheffl jscheffl approved these changes

bbovenzi Awaiting requested review from bbovenzi bbovenzi is a code owner

pierrejeambrun Awaiting requested review from pierrejeambrun pierrejeambrun is a code owner

ryanahamilton Awaiting requested review from ryanahamilton ryanahamilton is a code owner

shubhamraj-git Awaiting requested review from shubhamraj-git shubhamraj-git is a code owner

ashb Awaiting requested review from ashb ashb is a code owner

Assignees

No one assigned

Labels

area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

xcom_push raises 422 error when value is set to None

8 participants