[ http://jira.codehaus.org/browse/WAGONHTTP-5?page=all ]
Brett Porter closed WAGONHTTP-5.
--------------------------------
Resolution: Fixed
Fix Version/s: 1.0-beta-2
good point. fixed by adding an || timestamp == 0 on the check so that will
force a download.
> The getIfNewer method fails to work if file doesn't exist locally and the
> Last-Modified header isn't sent by the server
> -----------------------------------------------------------------------------------------------------------------------
>
> Key: WAGONHTTP-5
> URL: http://jira.codehaus.org/browse/WAGONHTTP-5
> Project: wagon-http
> Issue Type: Bug
> Affects Versions: 1.0-alpha-3
> Reporter: Kohsuke Kawaguchi
> Assigned To: Brett Porter
> Fix For: 1.0-beta-2
>
>
> The code doesn't work correctly if the following two conditions are met
> simultaneously:
> (i) the local file doesn't exist --- hence the timestamp parameter is 0
> (ii) the remote server doesn't send the "Last-Modified" header.
> Since the lastModified variable is initialized to 0 in line 355, if the above
> two conditions are met,
> the following if statement at line 371 evaluates to false:
> * if ( timestamp < lastModified )
> {
> retValue = true;
> and therefore the file won't be downloaded, causing the dependency to fail.
> This used to work with Maven 1.0.2.
> To fix this problem, initialize the lastModified variable to 1.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
