You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "J (JIRA)" <ji...@apache.org> on 2016/04/06 23:46:25 UTC

[jira] [Commented] (NET-591) FTPFileStampParserImpl failing when parsing correctly formed datetimestring (daylight saving issue)

    [ https://issues.apache.org/jira/browse/NET-591?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15229199#comment-15229199 ] 

J commented on NET-591:
-----------------------

Heres more on the problem
http://blog.tier1app.com/2013/05/29/daylight-savings-time-problem-java/

> FTPFileStampParserImpl failing when parsing correctly formed datetimestring (daylight saving issue)
> ---------------------------------------------------------------------------------------------------
>
>                 Key: NET-591
>                 URL: https://issues.apache.org/jira/browse/NET-591
>             Project: Commons Net
>          Issue Type: Bug
>          Components: FTP
>         Environment: locale: sv_SE
> timezone: "Europe/Stockholm"
> llinux/unix
>            Reporter: J
>
> In the class FTPTimeStampParserImpl,
> the SimpleDateFormat hackFormatter
> will throw an exception for certain datestrings, even though their SimpleDateFormat pattern for the hackFormatter instance is correct ("MMM d HH:mm yyyy")
> when in an environment that has daylight savings, like the environment reported in this bug report.
> For example:
> Line 123:          
>  parsed = hackFormatter.parse(timeStampStrPlusYear, pp);
> where timeStampStrPlusyear = "Mar 27 02:02 2016"
> will fail but 
>  timeStampStrPlusyear = "Mar 27 03:02 2016"
> would work.
> Looking around I found two suggestions that might fix these kind of problems in the code.
> 1) The default constructor of the Gregorian Calendar uses the local timezone of the machine. Setting the GregorianCalendar(TimeZone) constructor and pass UTC into that should work, but that would probally mean changing a lot of code.
> 2) Setting the hackformatters lenient to true will also make it work.
> However I don't know what's the right way for this project, one of the above or another solution.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)