[checkmk-commits] [tribe29/checkmk] 159dd0: pushover: minor refactoring

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 159dd015cbb4d871f2ecae2788d8737742687dac https://github.com/tribe29/checkmk/commit/159dd015cbb4d871f2ecae2788d873774… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-07-01 (Mon, 01 Jul 2019) Changed paths: M cmk/notification_plugins/pushover.py Log Message: ----------- pushover: minor refactoring * rename variables s and r * inline r_status Change-Id: Ie64e1e4122d8d979eb93ff84d03187593eab493f Commit: 7eaaa3ea298d8fa8b768254f1ee19ea48c35c4fd https://github.com/tribe29/checkmk/commit/7eaaa3ea298d8fa8b768254f1ee19ea48… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2019-07-01 (Mon, 01 Jul 2019) Changed paths: A .werks/7429 M cmk/notification_plugins/pushover.py M notifications/pushover Log Message: ----------- 7429 FIX pushover: fix broken error handling Since its introduction in version 1.2.8 the error handling of the pushover notification script is broken. Notifications were always handled as successful even if the Pushover API returned an error. Version 1.5.0 introduced another bug that printed an error message to the notify.log for successful notifications. Change-Id: I6801451fdc6b09398f147cba3174ae38aa86b844 Compare: https://github.com/tribe29/checkmk/compare/952511296d22...7eaaa3ea298d