This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/master by this push:
new 18b2c6d Removes unnecessary function call (#15956)
18b2c6d is described below
commit 18b2c6df36937fe6e57b5ec6a672e38c10837b57
Author: Jed Cunningham <66968678+jedcunning...@users.noreply.github.com>
AuthorDate: Thu May 20 09:09:57 2021 -0600
Removes unnecessary function call (#15956)
No need to make this call, as if no perms are passed
`sync_resource_permissions` short circuits anyways.
---
airflow/www/app.py | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/airflow/www/app.py b/airflow/www/app.py
index 632e1d2..a0bf447 100644
--- a/airflow/www/app.py
+++ b/airflow/www/app.py
@@ -61,9 +61,7 @@ def sync_appbuilder_roles(flask_app):
# will add the new Views and Menus names to the backend, but will not
# delete the old ones.
if conf.getboolean('webserver', 'UPDATE_FAB_PERMS'):
- security_manager = flask_app.appbuilder.sm
- security_manager.sync_roles()
- security_manager.sync_resource_permissions()
+ flask_app.appbuilder.sm.sync_roles()
def create_app(config=None, testing=False):
