Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTTP 500 Error when triggering DAG with invalid parameters #45896

Closed
2 tasks
missaarohi opened this issue Jan 22, 2025 · 2 comments
Closed
2 tasks

HTTP 500 Error when triggering DAG with invalid parameters #45896

missaarohi opened this issue Jan 22, 2025 · 2 comments
Labels
area:API Airflow's REST/HTTP API area:core kind:bug This is a clearly a bug won't fix

Comments

@missaarohi
Copy link

missaarohi commented Jan 22, 2025

Apache Airflow version

main (development)

If "Other Airflow 2 version" selected, which one?

No response

What happened?

When attempting to trigger a DAG through the stable API with invalid parameters, the API returns an HTTP 500 Internal Server Error. The error message shows 'Uuups'. This happens due to missing validation for the parameters before triggering the DAG.

What you think should happen instead?

The API should return a 400 Bad Request with appropriate error details instead of a 500 error.

How to reproduce

  1. Use the stable API to trigger a DAG.
  2. Send invalid parameters (e.g., missing required fields or incorrect values).
  3. Observe the returned HTTP 500 error.

Operating System

Ubuntu 20.04

Versions of Apache Airflow Providers

apache-airflow-providers-google==5.0.0

Deployment

Official Apache Airflow Helm Chart

Deployment details

No response

Anything else?

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@missaarohi missaarohi added area:core kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet labels Jan 22, 2025
Copy link

boring-cyborg bot commented Jan 22, 2025

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

@dosubot dosubot bot added the area:API Airflow's REST/HTTP API label Jan 22, 2025
@potiuk
Copy link
Member

potiuk commented Jan 22, 2025

Won't fix - we are rewriting the APIs in Airlfow 3 with FastAPI and Pydantic validation - so validation will be built in. There are no plans to enhance the behaviour and make any changes to the API framework used in Airflow 2.

@potiuk potiuk closed this as completed Jan 22, 2025
@potiuk potiuk added won't fix and removed needs-triage label for new issues that we didn't triage yet labels Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:API Airflow's REST/HTTP API area:core kind:bug This is a clearly a bug won't fix
Projects
None yet
Development

No branches or pull requests

2 participants