Merge pull request #1071 from Netflix/notif-fix
Fix cloned notifications
This commit is contained in:
commit
412757b178
|
@ -45,11 +45,14 @@ class CertificateCreationSchema(CertificateSchema):
|
||||||
@post_load
|
@post_load
|
||||||
def default_notification(self, data):
|
def default_notification(self, data):
|
||||||
if not data['notifications']:
|
if not data['notifications']:
|
||||||
notification_name = "DEFAULT_{0}".format(data['owner'].split('@')[0].upper())
|
data['notifications'] += notification_service.create_default_expiration_notifications(
|
||||||
data['notifications'] += notification_service.create_default_expiration_notifications(notification_name, [data['owner']])
|
"DEFAULT_{0}".format(data['owner'].split('@')[0].upper()),
|
||||||
|
[data['owner']],
|
||||||
notification_name = 'DEFAULT_SECURITY'
|
)
|
||||||
data['notifications'] += notification_service.create_default_expiration_notifications(notification_name, current_app.config.get('LEMUR_SECURITY_TEAM_EMAIL'))
|
data['notifications'] += notification_service.create_default_expiration_notifications(
|
||||||
|
'DEFAULT_SECURITY',
|
||||||
|
current_app.config.get('LEMUR_SECURITY_TEAM_EMAIL')
|
||||||
|
)
|
||||||
return data
|
return data
|
||||||
|
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue