As Julien says, the latest version does not fix this issue, and this is blocking deplyment (workaround: do it manually)
Also, the patch contains empty catch blocks (when trying to set the permissions on the FTP server).
In revision 524156 (line 193)
catch ( IOException e )
{
// TODO: handle
// michal I am not sure what error means in that context
// I think that we will be able to recover or simply we will fail later on
}
Maybe the catch should log a warning message?
And I think you can refine the catch block (from http://commons.apache.org/net/apidocs/org/apache/commons/net/ftp/FTPClient.html#completePendingCommand())
catch (FTPConnectionClosedException e)
{
// If the FTP server prematurely closes the connection as a result
// of the client being idle or some other reason causing the server
// to send FTP reply code 421. This exception may be caught either
// as an IOException or independently as itself.
}
catch ( IOException e )
{
// TODO: handle
// michal I am not sure what error means in that context
// I think that we will be able to recover or simply we will fail later on
}
Improvement
Closed
Major
my pom.xml
<build>
<defaultGoal>install</defaultGoal>
<extensions>
<extension>
<groupId>org.apache.maven.wagon</groupId>
<artifactId>wagon-ftp</artifactId>
</extension>
</extensions>
</build>
when exec: mvn site:deploy , error