From 2aa88dcb80f9d7cae4a42f3e435720412025e633 Mon Sep 17 00:00:00 2001 From: Jimmy Zelinskie Date: Fri, 29 Apr 2016 15:42:25 -0400 Subject: [PATCH] only send notifications when superusers enabled --- data/model/service_keys.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/data/model/service_keys.py b/data/model/service_keys.py index 193a25a84..382433da3 100644 --- a/data/model/service_keys.py +++ b/data/model/service_keys.py @@ -60,10 +60,11 @@ def _notify_superusers(key): if key.expiration_date is not None: notification_metadata['expiration_date'] = timegm(key.expiration_date.utctimetuple()) - superusers = User.select().where(User.username << config.app_config['SUPER_USERS']) - for superuser in superusers: - create_notification('service_key_submitted', superuser, metadata=notification_metadata, - lookup_path='/service_key_approval/{0}/{1}'.format(key.kid, superuser.id)) + if len(config.app_config['SUPER_USERS']) > 0: + superusers = User.select().where(User.username << config.app_config['SUPER_USERS']) + for superuser in superusers: + create_notification('service_key_submitted', superuser, metadata=notification_metadata, + lookup_path='/service_key_approval/{0}/{1}'.format(key.kid, superuser.id)) def create_service_key(name, kid, service, jwk, metadata, expiration_date, rotation_duration=None):