Review Request 111016: Set modified timestamp when moving resource from ftp -> file

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111016/#review34400
-----------------------------------------------------------


This review has been submitted with commit 4a75f8b6dbc36a858b5a05e4f65d39d07b0dc98c by Dawit Alemayehu to branch KDE/4.10.

- Commit Hook


On June 14, 2013, 5:04 a.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111016/
> -----------------------------------------------------------
> 
> (Updated June 14, 2013, 5:04 a.m.)
> 
> 
> Review request for kdelibs and David Faure.
> 
> 
> Description
> -------
> 
> This attached patch sets the modified timestamp on files moved from ftp servers to local file. Unfortunately, doing the same the other way around is a complete mess. That is because setting timestamp on FTP servers is anything but straight forward and there are 3 ways you can do it depending on the particular FTP server or how it is configured!
> 
> Here are the three commands:
> 
> 1.) MDTM command (not recommended). Using this command to set the modified timestamp completely goes against the RFC that defines it, RFC 3695. However, that does not stop some servers like Microsoft's and vsftpd from doing just exactly that!
> 
> 2.) MFMT command (recommended). This command was defined to do exactly what MDTM was not! Set the modified timestamp of a file on the FTP server. Though it never made it into a standards track RFC, it was defined in a draft IETF standard and seems to have been implemented by several FTP servers.
> 
> 3.) SITE UTIME command (????). This command seems to have originated in NcFtpd and later adapted by other FTP servers for compatibility sake. 
> 
> Anyhow, I am sure it is clear from the above listings that this can be a nightmare to deal with. Most client ftp software seem to simply implement support for all three and use the one the server supports by checking its response to the FEAT command. Until we do something similar, this patch will have to do.
> 
> 
> This addresses bug 55804.
>     http://bugs.kde.org/show_bug.cgi?id=55804
> 
> 
> Diffs
> -----
> 
>   kioslave/ftp/ftp.cpp b0868d8 
> 
> Diff: http://git.reviewboard.kde.org/r/111016/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130615/92af64ad/attachment.htm>