dheerajturaga commented on PR #72908: URL: https://github.com/apache/airflow/pull/72908#issuecomment-5686922628
Thanks for the fix. Could you please address these before merging? - Rerun or investigate the failing documentation CI job so the PR is green. - `_logfile_path()` is also used by `push_logs()`, so this change makes missing task instances return 404 from that endpoint too. Please document 404 for the push route, regenerate the OpenAPI spec, and add a regression test—or scope the 404 handling to the GET route only. --- Drafted-by: Codex (GPT-5); reviewed by @dheerajturaga 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]
