If you want this very nice plugin works for virtualmin vhosts, you have to replace lines :

define(‘LOG_PATH’, “/var/log/apache2/”);
by
define(‘LOG_PATH’, “/var/log/virtualmin/”);

and

exec(sprintf(“find %s -iname *_access.log”, LOG_PATH), $files);
by
exec(sprintf(“find %s -iname *_access_log”, LOG_PATH), $files);

in plugin source files.

it should work…

Fred

I was aware of the issue, just not got round to fixing it!

I will do.

When the log file it finds is empty it results in a divide by 0 error in the sub and complains in the munin logs. I have a couple of virtual hosts that get configured but not accessed until the client uploads web pages. It causes lots of errors in the munin logs for these scripts.

I fixed this on my sever with a dirty hack that if the value of $age was 0, return 0 and don’t execute :

return round($num_lines/$ago, 4);

Dunno if there is a more elegent fix for this. But I felt I should let you know.

Thanks again for this plugin. It’s awesome!