[Patches] [Patch #101187] Patch for ftplib to support REST

[noreply@sourceforge.net]

Patch #101187 has been updated. 

Project: 
Category: Modules
Status: Open
Summary: Patch for ftplib to support REST

Follow-Ups:

Date: 2000-Aug-15 22:34
By: tim_one

Comment:
Assigned to Barry for an opinion.  This came in after feature freeze, but it's arguably more of a bugfix than a new feature.  If you're capable of making that decision with more confidence than me, please Postpone it or assign it to a competent (not me) reviewer for 2.0.
-------------------------------------------------------

Date: 2000-Aug-22 07:09
By: moshez

Comment:
This seems great, and I'm very +1, except for one thing: the ``self.sendcmd("REST %.0f" % rest)'' thing: wouldn't it be better to self.sendcmd("REST %d" % int(rest))? 
Other then that, it seems to do nothing when the rest argument is not given, so it can't hurt too much. Oh, and of course, this needs doc patches.

 
-------------------------------------------------------

Date: 2000-Aug-22 15:33
By: none

Comment:
I realized some problem yesterday ... ftp-daemons which do not support REST throw a 502 exception
-------------------------------------------------------

Date: 2000-Aug-22 16:05
By: preisl

Comment:
Ok, forget my last commet (yep, it was me, who forgot to login) ... I think it's ok that the ftplib throws the exception, since the user knows, that he mustn't use the rest parameter then.
-------------------------------------------------------

Date: 2000-Aug-24 14:59
By: moshez

Comment:
OK, here's a stab at what's needed for documentation changes:

Add to the prototype of the "ntransfercmd()" and "transfercmd" an \optional{, rest} and to the description a paragraph:

If the \var{rest} parameter is given, it should be an integer, specifying how many initial bytes of the file the tranfer command should ignore. If the FTP server does not support the \code{REST} command, and the \var{rest} argument is used, an exception will be raised. Programs might want to deal with it by initiating a transfer without \var{rest}, and ignore the first \var{rest} bytes. (Of course, this option is much more costly in terms of time and network resources used.)
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=101187&group_id=5470