The GitHub Actions job "Tests" on airflow.git/explict-404-for-some-old-paths 
has failed.
Run started by GitHub user kaxil (triggered by kaxil).

Head commit for run:
99670710b1242dde48e68ad80753219d811e29fd / Ash Berlin-Taylor <a...@apache.org>
Return explicit 404 errors for `/helath` and `/api/v1/*`

THe current behaviour would be for this to hit the catchall route and serve up
the index.html for the SPA, but for these routes explicitly we don't want
that.

- `/health` was in Airflow 2 and was commonly used for Kube healthchecks --
  we want this to fail as it should be hitting the new URL instead
- `/api/v1` similarly, we want an explicit 404 for anything still hitting the
  old API rather than the confusion of getting HTML back when the client
  likely expects JSON.

I have also moved the catchall route from `/api/v2/` to `/api/`, as anything
under `/api/` is, well, an API and returning a 404 there is better.

This also corrects a small mistake in the API doc generation where we were
setting _all_ endpoints to include_in_schema=True, including the /api/v2/
catch all. Hence the change in the generated code

Report URL: https://github.com/apache/airflow/actions/runs/14471728042

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: jobs-unsubscr...@airflow.apache.org
For additional commands, e-mail: jobs-h...@airflow.apache.org

Reply via email to