Testing FTP (Was: Re: [PHP-QA] 2002-01-01 CVS ChangeLog Summary)

On Thu, Jan 03, 2002 at 12:24:25AM -0700, Zak Greant wrote : > First summary of the code added in the new year. :) > > +2002-01-01 Markus Fischer <mfischer@guru.josefine.at> > + * ext/ftp/php_ftp.c > + ext/ftp/php_ftp.h: > + - Switched to zend_parse_parameters(), unified error/warning messages, > + use real resources instead of integers, adjusted prototypes (hope I got > + them all). > + > + /php4/ext/ftp ftp.c ftp.h php_ftp.c php_ftp.h > + - Added ftp_set_option(), ftp_get_option() and support for setting a > + custom timeout. > > Some cool fixes and extensions to the FTP extension. Markus made the code > both shorter and better. :) Again, another thing to watch for in testing. > > We should also make sure to test the new ftp_set_option() and > ftp_get_option(), as well as the timeout functionality. I don't know if you guys ever noticed, but while converting and testing I found out the the FTP extension still has some quirks. For exmaple, listing the root directory of the remote ftp server with '/' sometimes doesn't work, or you need multiple slashes like '//'. I also expirienced a complete hang when putting files to the remote server. I only could verify the hang vom linux to linux box when the file was already PUT but the extension was waiting for an answer. The hang usually always takes 90 seconds because of the default timeout. The good news now is you can alter this timeout to one second and the thing gets less worse ... but doesn't solve the original problem. Well, I didn't expirienced those things from win32 to the same linux ftp server ... I'm investigating this myself too, but other testers are always a good idea. - Markus ps: Zak, you're summaries are really a great thing! -- Please always Cc to me when replying to me on the lists.