potiuk commented on PR #70889: URL: https://github.com/apache/airflow/pull/70889#issuecomment-5173304281
Both threads addressed and resolved. Mocks now carry `spec`. I did not move `backfill_id` into the dependency signature: the dependency is also attached to `GET /backfills`, `POST /backfills` and `POST /backfills/dry_run`, which have no `backfill_id` path parameter, so FastAPI would resolve it as a *query* parameter there — 422-ing those routes if declared bare, or exposing `?backfill_id=` in the public schema and letting a caller steer authorization at a backfill the handler never touches. Detail in the thread; a shared type alias is the alternative if drift is the concern. @amoghrajesh @pierrejeambrun --- Drafted-by: Claude Code (Opus 5); reviewed by @potiuk before posting -- 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]
