Regarding FastCommonAccessLogValve and ExtendedAccessLogValve:

FastCommonAccessLogValve:

>> There is a bug, that sneaked in rev.781753
>> http://svn.apache.org/viewvc?rev=781753&view=rev
>> That is:
>> the timeZoneNoDST field is never assigned and remains null
>> It was ok in 5.5.27.
>
> That is an error in my backport. That line should never have been deleted. 
> I'll
> fix that now.

That backport should be applied to FastCommonAccessLogValve as well.
The bug #46982 is not fixed there yet.


ExtendedAccessLogValve:

The patch for ExtendedAccessLogValve is broken.

See the getDate() function there.
"currentDate.getTime()" cannot be called unless this Date instance is
local to this thread, but it is not there.  Also, that Date object is
returned by that function and can be used by other threads while
getDate() modifies it.


Best regards,
Konstantin Kolinko

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to