Skip to content

Fix failing DAG Astro links for API v2#241

Merged
redreceipt merged 1 commit intomainfrom
sym-45-i-clicked-a-failed-dag-link-and-got-an-error
Mar 18, 2026
Merged

Fix failing DAG Astro links for API v2#241
redreceipt merged 1 commit intomainfrom
sym-45-i-clicked-a-failed-dag-link-and-got-an-error

Conversation

@redreceipt
Copy link
Member

Summary

  • normalize the Astro UI base URL by stripping any /api/v<version> suffix from AIRFLOW_API_BASE_URL
  • keep failed DAG run links pointed at the Astro UI even when the Airflow API base uses /api/v2
  • add regression coverage for both v1 and v2 API suffixes

Why

Failed DAG links were built from the Airflow API base URL and only handled /api/v1. With the current /api/v2 config, the dashboard linked users to an API route instead of the Astro UI and returned {"error":"API route not found"}.

Proof

None

@linear
Copy link

linear bot commented Mar 18, 2026

SYM-45

@redreceipt redreceipt temporarily deployed to bug-board-sym-45-i-clic-yqkbda March 18, 2026 01:43 Inactive
@redreceipt redreceipt marked this pull request as ready for review March 18, 2026 01:48
@redreceipt redreceipt merged commit a8b85cc into main Mar 18, 2026
6 checks passed
@redreceipt redreceipt deleted the sym-45-i-clicked-a-failed-dag-link-and-got-an-error branch March 18, 2026 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant