ashb commented on a change in pull request #21472:
URL: https://github.com/apache/airflow/pull/21472#discussion_r807058105
##########
File path: airflow/www/extensions/init_jinja_globals.py
##########
@@ -66,9 +66,13 @@ def prepare_jinja_globals():
'airflow_version': airflow_version,
'git_version': git_version,
'k8s_or_k8scelery_executor': IS_K8S_OR_K8SCELERY_EXECUTOR,
- 'rest_api_enabled': conf.get('api', 'auth_backend') !=
'airflow.api.auth.backend.deny_all',
+ 'rest_api_enabled': False,
Review comment:
Hmmm, this one isn't right.
This is used to control if the "Test Connection" button is enabled for the
Connections form - see https://github.com/apache/airflow/pull/19792
So we probably want to test if `session` is in this list.
##########
File path: airflow/www/extensions/init_jinja_globals.py
##########
@@ -66,9 +66,13 @@ def prepare_jinja_globals():
'airflow_version': airflow_version,
'git_version': git_version,
'k8s_or_k8scelery_executor': IS_K8S_OR_K8SCELERY_EXECUTOR,
- 'rest_api_enabled': conf.get('api', 'auth_backend') !=
'airflow.api.auth.backend.deny_all',
+ 'rest_api_enabled': False,
Review comment:
Read the code Ash :faceplam:
--
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]
