sni / Thruk

Thruk is a multibackend monitoring webinterface for Naemon, Nagios, Icinga and Shinken using the Livestatus API.
http://www.thruk.org
Other
409 stars 149 forks source link

Upgrade to 3.00 broke history via Livestatus #1232

Closed dlucredativ closed 1 year ago

dlucredativ commented 1 year ago

Our 2.50 installation was able to display host and service history, fetching it via livestatus. After an upgrade to 3.00 we get Unexpected escape character errors. It seems to me that this is due to e8078603f6e7b36b0574a860d29912481dcd77c9, by which the history is no longer served via history.pm but showlog.pm. The former controller did not inject ~~ filters, while the latter does in the absense of a logcache. Thruk treats regexes as PCRE while livestatus expects POSIX, hence \s and quotemeta-results are mostly unusable.

sni commented 1 year ago

do you have an example hostname or service name which breaks the page?

dlucredativ commented 1 year ago

A service_description that contains a whitespace or a hyphen for example.

sni commented 1 year ago

what kind of backend is this? I tried hosts/services with spaces, with and without logcache and it works as expected. (Tested with naemon-core and naemon-livestatus)

dlucredativ commented 1 year ago

I think you can close this as «wontfix». The official Checkmk packages are built against RE2 and don't have this problem. We will have to rebuild our livestatus instance.

sni commented 1 year ago

alright, thanks for the feedback