Bug #120906 for Log-Log4perl: timezone incorrect after daylight saving time switch

Mon Apr 03 11:43:57 2017 abraxxa [...] cpan.org - Ticket created

Subject:

timezone incorrect after daylight saving time switch

Using this timestamp field pattern %d{yyyy-MM-ddTHH:mm:ss.SSSZ} the timezone information is still +0100 instead of +0200 after the daylight saving time switch. From reading the code I saw that the localtime perl function is used by default and there seems to be no way to specify that UTC should be returned in which case the literal 'Z' could be used to indicate that it's UTC. On restart of the app the timezone information is correct. This is perl 5, version 22, subversion 1 (v5.22.1) built for x86_64-linux

Mon Apr 03 11:47:13 2017 abraxxa [...] cpan.org - Correspondence added

Am Mo 03. Apr 2017, 11:43:57, ABRAXXA schrieb: Show quoted text

> From reading the code I saw that the localtime perl function is used > by default and there seems to be no way to specify that UTC should be > returned in which case the literal 'Z' could be used to indicate that > it's UTC.

I have to revert this statement. 'gmtime' can be used instead of localtime if the global module variable $GMTIME is set to a true value but this isn't documented and would also cause action at distance if one module enables the variable whereas another expects localtime to be used.

Mon Nov 06 12:23:41 2017 ether [...] cpan.org - Correspondence added

On 2017-04-03 08:43:57, ABRAXXA wrote: Show quoted text

> Using this timestamp field pattern %d{yyyy-MM-ddTHH:mm:ss.SSSZ} the > timezone information is still +0100 instead of +0200 after the > daylight saving time switch.

I ran into this yesterday (when switching from daylight savings back to standard time), in the 'America/Los_Angeles' timezone. The offset is calculated only once, at startup, in Log::Log4perl::DateFormat::rep, by calculating gmtime - localtime. Even though this offset can change during runtime, the cached value is always used. I would suggest perhaps switching to using POSIX::strftime to do the formatting, which uses the system's (fast) routines, rather than trying to optimize in custom perl code.

Mon Nov 06 12:23:42 2017 The RT System itself - Status changed from 'new' to 'open'

Thu Dec 07 13:41:18 2017 ether [...] cpan.org - Severity Important added