On Sat, 5 Jun 2004, Jeff Trawick wrote:
[EMAIL PROTECTED] wrote:
The patch here: <http://drbacchus.com/files/access_logging_patch.txt> for 2.0.49 creates logs that look like this:
[Fri May 28 22:19:16 2004] [notice] Apache/2.0.49 (Unix) configured -- resuming normal operations [Fri May 28 22:19:24 2004] [error] [client 192.168.1.33] client denied by order directive at line 362 of /opt/apache2/conf/httpd.conf for: /opt/apache2/htdocs/bar
What about enabling via a per-dir directive? Looks very useful, but some sites might have a significant number of new entries in error log serving no useful purpose (for them).
The problem with per-dir is that the user might not know exactly where in the config file he should be editting. If they did, they probably wouldn't have a problem finding the relevant allow/deny in the first place.
What about keeping the old error message, but logging an *additional* message at loglevel debug with the additional info?
Two other notes: - Personally, I hate arbitrary path-size limitations. They always bite you when you least expect it. What about dynamically sizing this? - You'll get more people to read your patches if you use "diff -u".
I somehow misread the patch and didn't remember what was logged before the patch. I withdraw any and all comments about not logging the config location by default.
