cvs commit: src/lib/libfetch fetch.3 ftp.c

Nate Lawson nate at root.org
Wed May 9 16:38:24 UTC 2007


Dag-Erling Smørgrav wrote:
> Nate Lawson <nate at root.org> writes:
>> Could it be possible to check the values from the URL used to derive
>> TYPE for RETR and then if it's the same as the value for MDTM, don't
>> reissue it?  Same for MODE.
> 
> I suggest you look at the code.  We *do* derive the TYPE value from
> the URL.  We do *not* trust the server state to be consistent from one
> request to another.  MDTM and RETR are two separate requests.

Sorry, my message might have been unclear.  I've looked at the code.

> We *could* change the code to remember the last TYPE and MODE it
> issued for the current server, and not issue new ones unnecessarily,
> but we're quickly getting to the point where the FTP code needs a
> major cleanup, at least when it comes to connection caching.

This is what I was suggesting.  I'll look into patching the current code
to do this.  If it's too hard, I'll skip it.

-- 
Nate


More information about the cvs-src mailing list