potiuk commented on code in PR #56564:
URL: https://github.com/apache/airflow/pull/56564#discussion_r2427520572
##########
airflow-core/src/airflow/api_fastapi/common/parameters.py:
##########
@@ -767,7 +767,7 @@ def _transform_dag_run_states(states: Iterable[str] | None)
-> list[DagRunState
return [None if s in ("none", None) else DagRunState(s) for s in
states]
except ValueError:
raise HTTPException(
- status_code=status.HTTP_422_UNPROCESSABLE_ENTITY,
+ status_code=status.HTTP_422_UNPROCESSABLE_CONTENT,
Review Comment:
It awfully looks like that from the error wiht lowest deps. But likely it's
starlette that should be bumped.
```
FAILED
airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dag_run.py::TestGetDagRuns::test_invalid_state
- AttributeError: module 'starlette.status' has no attribute
'HTTP_422_UNPROCESSABLE_CONTENT'. Did you mean: 'HTTP_422_UNPROCESSABLE_ENTITY'?
FAILED
airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_task_instances.py::TestGetTaskInstances::test_bad_state
- AttributeError: module 'starlette.status' has no attribute
'HTTP_422_UNPROCESSABLE_CONTENT'. Did you mean: 'HTTP_422_UNPROCESSABLE_ENTITY'?
================================================================== 2 failed,
1940 p
```
Yep. Added in 0.48.0 so we need to >=0.48.0 on starlette.
--
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]
