github-actions[bot] opened a new pull request, #65583:
URL: https://github.com/apache/airflow/pull/65583
* Fix UI log fetch crash when ti.hostname is empty
When tasks fail to report their hostname (e.g., due to cellular worker proxy
overrides dropping the internal API patched payload), the TaskInstance hostname
defaults to empty. This causes _get_log_retrieval_url to construct an invalid
URL ('http://:8793/...') which completely crashes log serving with an 'Invalid
URL... No host supplied' error from python requests.
closes: #64263
* Fix UI log fetch crash when ti.hostname is empty: address reviewer feedback
* Fix mypy static checks and add hostname_callable config help
* Update airflow-core/src/airflow/utils/log/file_task_handler.py
Co-authored-by: Jason(Zhe-You) Liu
<[email protected]>
---------
(cherry picked from commit 7c3eacc3cfb9104128def18faed221fd4f4b2cd3)
Co-authored-by: Gourav N S S <[email protected]>
Co-authored-by: Jason(Zhe-You) Liu
<[email protected]>
--
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]
