On Fri, 28 Dec 2007 19:31:31 -0000 [EMAIL PROTECTED] wrote: > @@ -140,11 +143,15 @@ > http://svn.apache.org/viewcvs.cgi?rev=607245&view=rev > Backport version for 2.2.x of patch: > Trunk version of patch works > - +1: rpluem, niq > + +1: rpluem, niq, wrowe > niq: Provisional +1, but the error logging should be at a > consistent level (maybe WARNING?) > rpluem: Set it to ERROR in all cases as IMHO this should not > happen. If this level is too high we can reduce it later. > + wrowe: disagree with rpluem - it's incredibly disruptive to > admins > + to have their logs filled with noise - warning would be > ok, > + provided there's no more than one entry per failed > request. > + If their request would die outright, only then is > rpluem right.
+1 to wrowe's comment. I was thinking the same, just not loudly enough to be the first to say so. -- Nick Kew Application Development with Apache - the Apache Modules Book http://www.apachetutor.org/
