[checkmk-commits] [tribe29/checkmk] 2fe5a4: pushover: minor refactoring

Branch: refs/heads/1.6.0 Home: https://github.com/tribe29/checkmk Commit: 2fe5a423a09a961de469f4264ad6bfceb3203ade https://github.com/tribe29/checkmk/commit/2fe5a423a09a961de469f4264ad6bfceb… 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: 1583408f7cf25e2dff2d97f8c31f608da7f17df0 https://github.com/tribe29/checkmk/commit/1583408f7cf25e2dff2d97f8c31f608da… 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/210a99ca734f...1583408f7cf2