livestatus: Stop relying on broken short-cut65/565/1
commit7e6e2ae5c60a3b70214599caa3d57e4a68e9f960
authorRobin Sonefors <robin.sonefors@op5.com>
Wed, 9 Apr 2014 07:54:00 +0000 (9 09:54 +0200)
committerRobin Sonefors <robin.sonefors@op5.com>
Wed, 9 Apr 2014 11:22:22 +0000 (9 13:22 +0200)
tree0f84efbc0a00db5769e71c47617d40171d129bb1
parent720209e813dfa8eca7627ec513ed3ae45f587b19
livestatus: Stop relying on broken short-cut

Earlier versions of the livestatus library had a short-cut, where you
could give a string as a column name, in which case the library would
assume you wanted less deeply nested arrays back - which as if by
accident happened to correspond to what we need whenever we want to get
a list of names.

This version of the livestatus library's support for stat responses
seemingly makes the shortcut appear to work, except the query building
code doesn't support the shortcut, so that instead of restricting the
columns in the query to what we'll be returning, we download ALL THE
DATAS! This has the downside of large installations eating up tons of
memory and crashing horribly whenever a stupidly written form is about
to be rendered.

This works around that "quirk" in a minimally invasive way, by no longer
relying on that shortcut in all the places I've found it to be used.

The PNP helper by design throws an exception, so it won't run in a
browser, but it showed up in my grep. The execute_command code
(bug #8617) should no longer be trivially triggerable, but was manually
verified when the same change was made for 6.2. The tests passes.

Signed-off-by: Robin Sonefors <robin.sonefors@op5.com>
application/helpers/nagvisconfig.php
application/helpers/pnp.php
application/models/execute_command.php
test/unit_test/tests/report_Test.php