From 4ba1008cf1a0af54d53d8c1e281de09255d0faa3 Mon Sep 17 00:00:00 2001 From: Hussein Awala Date: Thu, 31 Aug 2023 20:49:00 +0200 Subject: [PATCH] Remove useless string join from core --- airflow/api/auth/backend/kerberos_auth.py | 2 +- airflow/www/views.py | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/airflow/api/auth/backend/kerberos_auth.py b/airflow/api/auth/backend/kerberos_auth.py index 5306ca6d1ee74..e5458d05529c1 100644 --- a/airflow/api/auth/backend/kerberos_auth.py +++ b/airflow/api/auth/backend/kerberos_auth.py @@ -144,7 +144,7 @@ def decorated(*args, **kwargs): response = function(*args, **kwargs) response = make_response(response) if ctx.kerberos_token is not None: - response.headers["WWW-Authenticate"] = " ".join(["negotiate", ctx.kerberos_token]) + response.headers["WWW-Authenticate"] = f"negotiate {ctx.kerberos_token}" return response if return_code != kerberos.AUTH_GSS_CONTINUE: diff --git a/airflow/www/views.py b/airflow/www/views.py index add041c51d103..d3ecae5044962 100644 --- a/airflow/www/views.py +++ b/airflow/www/views.py @@ -561,7 +561,7 @@ def get_key_paths(input_dict): for key, value in input_dict.items(): if isinstance(value, dict): for sub_key in get_key_paths(value): - yield ".".join((key, sub_key)) + yield f"{key}.{sub_key}" else: yield key @@ -1528,16 +1528,16 @@ def rendered_templates(self, session): for key, value in content.items(): renderer = task.template_fields_renderers.get(key, key) if renderer in renderers: - html_dict[".".join([template_field, key])] = ( + html_dict[f"{template_field}.{key}"] = ( renderers[renderer](value) if not no_dagrun else "" ) else: - html_dict[".".join([template_field, key])] = Markup( + html_dict[f"{template_field}.{key}"] = Markup( "
{}
" ).format(pformat(value) if not no_dagrun else "") else: for dict_keys in get_key_paths(content): - template_path = ".".join((template_field, dict_keys)) + template_path = f"{template_field}.{dict_keys}" renderer = task.template_fields_renderers.get(template_path, template_path) if renderer in renderers: content_value = get_value_from_path(dict_keys, content)