Skip to content

Conversation

@vincbeck
Copy link
Contributor

HTTP_422_UNPROCESSABLE_ENTITY is deprecated and HTTP_422_UNPROCESSABLE_CONTENT should be used instead. That removes quite a few deprecation warnings when the api server starts


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@ashb
Copy link
Member

ashb commented Nov 28, 2025

FAILED airflow-core/tests/unit/cli/commands/test_task_command.py::TestCliTasks::test_cli_test_with_deferrable_operator - assert 'Found File /tmp/temporary_file_for_testing' in "DAG bundles loaded: dags-folder, example_dags\nRemoving ownership of team 'None' from Dag bundle 'dags-folder'\nRemov...NTENT'. Did you mean: 'HTTP_422_UNPROCESSABLE_ENTITY'?\n[DAG TEST] end task task_id=wait_for_file_async map_index=-1\n"

Looks like we'll need some level of compat here.

@vincbeck vincbeck force-pushed the vincbeck/422 branch 4 times, most recently from 48ac8d9 to e365c23 Compare November 28, 2025 23:22
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one nit, non blocking

@vincbeck vincbeck merged commit 9c4ebf3 into apache:main Dec 1, 2025
118 checks passed
@vincbeck vincbeck deleted the vincbeck/422 branch December 1, 2025 15:53
RoyLee1224 pushed a commit to RoyLee1224/airflow that referenced this pull request Dec 3, 2025
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
itayweb pushed a commit to itayweb/airflow that referenced this pull request Dec 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants