search

hudson-trading / netbox-nagios

Plugin to show Nagios host and service status in NetBox on the device page.
Other
44 stars 10 forks source link

Additional \n for icinga2 livestatus on query needed #5

Closed tweippert closed 3 years ago

tweippert commented 3 years ago

I use icinga2 and the livestatus implementation there needs an additional \n to getting data from icinga2.

Not sure it this breaks MK Livestatus on Nagios, as i don't use it, can't test it, but maybe you had the chance to retest if this fix for icinga2 needs some more "improvement", something like additional optional setting to add the final \n to the query?

Maybe this additional work is only needed if the change breaks Nagios/MK Livestatus!?

fabi125 commented 3 years ago

Thanks for the PR! I just tested this with Nagios and it still works.