The ftplib.FTP class currently allows the ability toprovide a callback function for the retrbinary &retrlines methods to invoke.  However, thecorresponding storbinary and storlines methods do not.My ReleaseForge application currently uses the ftplibto send files (rather than retrieve) and it's helpfulto know the percentage of the file that has been sent. For my app I subclassed ftplib.FTP and reimplementedstorbinary to optionally accept a callback function asa parameter.  However, I think this would be a useful method toinclude in the standard python distribution.  I'veattached a modified version of ftplib.py which addsthis functionality.  Unlike the retrX methods, thestorX methods only invoke the callback function if it'snot None so that it will not interfere with existingcode.  For completeness, the retrX methods invokeprint_line if a callback function is not provided anddoing so for the storX functions may be undesired.

Logged In: YES user_id=849994Please provide your patch as a diff between the originalftplib.py and your version. It greatly helps reviewers.

Logged In: YES user_id=420501Attached patch of ftplib.py (of Python 2.4.3)

I'd like to voice some support for this patch, if that's not out of line.  I've used it myself in at least one app, and it would be nice to have it as a standard feature in 2.6.It seems odd that the callback feature is in the two retrieval methods, but not the store methods.  Ruby, for example, supports it in both.  Not that Python has to do everything Ruby does, but there doesn't seem to be any obstacle to adding this support.

Terry, your comment is definitely *not* out of line!  The more people comment on patches the easier it is for us to make a decision.  It would be great to comment on other patches or bugs that affect you.  Thanks.

accepted, committed as svnr60188 for 2.6.  I also cleaned up the retr*and stor* doc strings.

Could I propose the following docstring corrections?@@ -313,7 +313,7 @@         expected size may be None if it could not be determined.          Optional `rest' argument can be a string that is sent as the-        argument to a RESTART command.  This is essentially a server+        argument to a REST(art) command.  This is essentially a server         marker used to tell the server to skip over any data up to the         given marker.         """@@ -403,7 +403,7 @@         """Retrieve data in line mode.  A new port is created for you.          Args:-          cmd: A RETR or LIST command.+          cmd: A RETR, LIST, NLST or MLSD command.           callback: An optional single parameter callable that is called                     for each line with the trailing CRLF stripped.                     [default: print_line()]

thanks.  applied (well, they will be when svn.python.org comes back)On 1/22/08, Giampaolo Rodola' <report@bugs.python.org> wrote:>>> Giampaolo Rodola' added the comment:>> Could I propose the following docstring corrections?>>

Another one. On line 542:-        # Note that the RFC doesn't say anything about 'SIZE'+        # SIZE command is defined in RFC-3659