[GitHub] [airflow] rino0601 commented on a diff in pull request #24373: Fix timezone display for logs on UI by adding %z for %(asctime)s (#23796)

[GitBox <gi...@apache.org>]

rino0601 commented on code in PR #24373:
URL: https://github.com/apache/airflow/pull/24373#discussion_r897653116


##########
airflow/config_templates/airflow_local_settings.py:
##########
@@ -60,10 +59,15 @@
     'version': 1,
     'disable_existing_loggers': False,
     'formatters': {
-        'airflow': {'format': LOG_FORMAT},
+        'airflow': {
+            'format': LOG_FORMAT,
+            'class': 'airflow.utils.log.timezone_aware.TimezoneAware',

Review Comment:
   @uranusjr 
   
   Like `COLORED_FORMATTER_CLASS`, I changed it to be user configurable. This is a bugfix, and backwards compatibility was adjusted in js, so a new class was designated as fallback.
   Could you check it out?



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org