Re: new ftp-module doesn't work at all (on solaris)
| From: | Mark Musone | Date: | Fri, 12 Nov 1999 00:27:34 +0000 |
| Subject: | Re: new ftp-module doesn't work at all (on solaris) | ||
| References: | 1 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-12317@lists.php.net to get a copy of this message | ||
if i'm not mistaken, a newline flushes a stream..
either way, andy's checking into it..
Mark
> Andrew,
> how well is this module tested? - i'm under teh impression that:
>
> ....
> int
> ftp_login(ftpbuf_t *ftp, const char *user, const char *pass)
> {
> if (ftp == NULL)
> return 0;
>
> fprintf(ftp->fp, "USER %s\r\n", user);
> if (!ftp_getresp(ftp))
> ....
>
> simply cannot work reliable, 'cause you *need* to make sure that your "USER.."
> gets flushed thru to the socket. (eg using fflush()).
>
> this is just a sample (flushes are missing all over the place).
>
> i'm also not sure if using stdio for the FTP protocol will be a reliable solution at all.
> using tis we'll never be able to handle time-outs etc.
>
> don't get me wrong: i really like the module as such - but in my opinion it needs more
> work. i was hoping i could use it to communicate to a weird FTP server that does not allow for STOR
> with a path-name....
>
>
> regards,
> tc
>
>
> --
> PHP Development Mailing List <http://www.php.net/>
> To unsubscribe, e-mail: php-dev-unsubscribe@lists.php.net
> For additional commands, e-mail: php-dev-help@lists.php.net
> To contact the list administrators, e-mail: php-list-admin@lists.php.net
>