riskersen / Monitoring

Monitoring plugins wich are Nagios/icinga compatible
65 stars 113 forks source link

check_printer: snmp2_walk behaviour changed since php5.2 #23

Closed hapm closed 8 years ago

hapm commented 8 years ago

check_printer outputs an internal error when running the alerts command since php v5.2.0.

Looks like they changed how snmp[|2|3]walk works in php5.2 and up: it doesn't return an associative array with oids as keys. There are new functions called [snmp2real_walk](http://php.net/manual/de/function.snmp2-real-walk.php), that work like snmp2_walk did before.

riskersen commented 8 years ago

Hi Markus,

Maybe you could submit with your proposed fix? Would be very nice! :-)

Best regards Oliver Am 29.06.2016 13:49 schrieb "Markus Andree" notifications@github.com:

check_printer outputs an internal error when running the alerts command since php v5.2.0.

Looks like they changed how snmp[|2|3]_walk works in php5.2 and up: it doesn't return an associative array with oids as keys. There are new functions called snmp2realwalk http://php.net/manual/de/function.snmp2-real-walk.php, that work like snmp2_walk http://php.net/manual/de/function.snmp2-walk.php did before.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/riskersen/Monitoring/issues/23, or mute the thread https://github.com/notifications/unsubscribe/AH7uzLvpSgCcoQK4O25IqUmdURxphHidks5qQlvbgaJpZM4JBDIX .

riskersen commented 8 years ago

Fixed in aacbf38 by @pinkra