cpliakas/nagiostatus

PHP parser library for the Nagios status.dat file.

nagiostatus's Issues

Implement pluggable parsers

Currently the parser assumes a 3.x config file. The parser should be pluggable so that alternate parsers can be leveraged to handle different Nagios versions in addition to future versions that haven't been released yet.

Improve performance of parsing

Current every line is evaluated by at lease one regular expressions. If we change the logic of the parser, the 80% case could be handled first (which is parsing reports inside a status block) and we can probably eliminate the regular expressions as well.

Implement the observer pattern so that loggers can be added

Pluggable logging functionalities that wrap around systems such as Zend_Log or Drupal's watchdog should be implemented by this library to capture errors and debugging information.

UPDATE: Implementing the observer pattern should work fine without explicitly adding the loggers.

Reconcile differences in JSON plugin

After the switch to the per-status rendering, the JSON plugin no longer works.

Reduce memory consumption

Currently the script will consume about 100MB of memory when parsing an 11MB file. This is due to creating an array from the parsed file as well as storing the entire document in memory prior to outputting. The code should be refactored so that the plugins can output the content directly so that no bog data structures are stored in memory.

Fix false positive in error detection

The buildDocument() method incorrectly identifies empty lines as being invalid. The logic should be modified to correctly handle empty lines. There is no error handling currently in place to the library still works, but it will cause problems when we implement the observer pattern.

Provide fluent interface

It'd be nice if the Parser provided a fluent interface so you could do:

<?php
  $document = new SimpleXMLElement(Nagiostatus_Parser::instance('/usr/local/nagios/var/status.dat')
    ->render('xml', true));
?>

You could also chain your registerPlugin() calls instead of making them static methods.

Come up with a more useful XML format

Currently the XML structure closely matches the status.dat structure, so it isn't very useful when used on conjunction with things like XPath. We should come up with a structure that makes it easy to navigate the file.