Logging Disabled about engintron HOT 5 CLOSED

To fix this (we confirmed this) should be relatively trivial;

In function "sync_vhosts" simply set the first access_log path to:

/usr/local/apache/domlogs/$DOMAIN

Then remove the remaining references (further down) to

access_log off

from engintron.

Here's the thing. Access logs are deliberately off for static files ONLY for 2 reasons: a) access logs become more meaningful and you don't get flooded with missing favicon.ico entries or other missing files and b) not having Nginx record access for such files makes Nginx operate a bit faster.

Besides, Nginx has its own access log which has nothing to do with Apache. Moreover, if we patch the access_log path to use Apache's (inside domlogs) then both Apache and Nginx will conflict with each other because they will be accessing and trying to write the same files!

from engintron.

EDIT: Removed responses. I'll just fork my own repo with fixes included to save the discussion and argument. Thanks for your time.

from engintron.

First off, Nginx is not a drop in replacement for Apache. Apache is still working but behind Nginx. So you should not be messing with Apache's logs imho. Nginx, like I said, has its own logs under /var/log/nginx - the error log specifically can be viewed via cPanel, from within Engintron.

Secondly, I'm definitely not confusing access logs with error logs. By disabling access logs for static files you are not hogging your access and error logs with entries of missing static files (which most of the times you won't care about). Just go ahead and request http://engintron.com/somefavicon.ico - this will never be recorded in that server's access or error logs. And this is a good thing for cPanel. You know why? Because the default /usr and /var partitions created by 99,99% of all cPanel installations are extremely small in size and this can lead to them filling up pretty quickly on busy sites and thus to an unstable server.

I did notice that the access_log directive was entirely off for each vhost and I've fixed that. But there is no point in logging static files.

from engintron.

Closed to save time and argument. I'll fork my own repo with these changes and fixes included.

from engintron.