Discovery Plugin - Bug report

[skirmish]

In discovery where there is more than 1 page of results, the next page link is invalid. The link in my setup reads:

http://xx.xx.xx.xx/cacti/plugins/discov ... us=&page=2

host.php should be discover.php, there is no page called host.php.

[gandalf]

This is which version of discovery, please?
R.

[straderw]

I am also having the same issue. I am running v1.1 it appears that in the discover.php file on line 323, 329, & 389 it clearly puts the a href as host.php.

[gandalf]

IMHO, this has been corrected in current SVN.
Oops, don't know if that's publicly available.
Then, please replace those occurences with "discover.php".
The fix looks like

Code: Select all

<strong><< "; if ($page > 1) { $nav .= "<a class='linkOverDark' href='" . htmlspecialchars("discover.php?view&status=$status&os=$os&snmp=$snmp&ip=$ip&host=$host" . "&page=" . ($page-1)) . "'>"; } $nav .= "Previous"; if ($page > 1) { $nav .= "</a>"; } $nav .= "</strong>then (to prevent URL injection)

R.

[straderw]

Well my fix right now was to make a symbolic link called host.php and pointed it to the discover.php.

Any idea when the fix might be released?

[gandalf]

This is the sole right of the author. In this case, this is "cigamit".
R.