Skip to content

Commit

Permalink
Don't send notifications that are marked inactive (Netflix#1015)
Browse files Browse the repository at this point in the history
Apparently previously Lemur ignored the "active" flag of notifications.
  • Loading branch information
intgr authored and kevgliss committed Dec 6, 2017
1 parent ad88637 commit e1f241b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lemur/notifications/messaging.py
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ def needs_notification(certificate):
days = (certificate.not_after - now).days

for notification in certificate.notifications:
if not notification.options:
if not notification.active or not notification.options:
return

interval = get_plugin_option('interval', notification.options)
Expand Down

0 comments on commit e1f241b

Please sign in to comment.