[PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2005-05-05 Thread Ilia Alshanetsky
iliaa Thu May 5 22:31:08 2005 EDT Modified files: /php-srcNEWS /php-src/ext/standard http_fopen_wrapper.c Log: Added max_redirects context option that specifies how many HTTP redirects to follow. http://cvs.php.net/diff.php/php-src/NEWS?r

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-10 Thread Wez Furlong
I started that with the curlstreams, but libcurl just didn't really want to play nicely as a stream. In addition, PHP kept on requiring the latest and greatest versions of libcurl for the curl extension; they are frequently not supported in stable binary distros and were hard to obtain (crappy SF.

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Adam Maccabee Trachtenberg
On Wed, 8 Sep 2004, Andi Gutmans wrote: > Yep I concur. No need to bloat the code base for no good reason. > Whoever needs the flexibility you are talking of can use CURL. Maybe another alternative would be to revive the cURL wrappers project? That would give us the benefits of being able to leve

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Rasmus Lerdorf
Why not integrate libcurl more intimately? I can see Sara's point in wanting to be able to pipeline http streams, but re-inventing the wheel doesn't seem like a good idea either. Get the streams code to use the appropriate features in libcurl, if present. We should be able to solve the problem w

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Andi Gutmans
Yep I concur. No need to bloat the code base for no good reason. Whoever needs the flexibility you are talking of can use CURL. Andi At 12:29 PM 9/8/2004 -0700, Sterling Hughes wrote: > Sacha- > > I can appreciate that there is a limited potential need for PHP to support > chunked encoding >righ

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Marcus Boerger
Hello Sascha, Wednesday, September 8, 2004, 9:34:51 PM, you wrote: >> I can appreciate that there is a limited potential need for PHP to support >> chunked encoding >right-now<, however this is one step in a larger process > I suggest you complete the larger process of cloning curl and >

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Sascha Schumann
I can appreciate that there is a limited potential need for PHP to support chunked encoding >right-now<, however this is one step in a larger process I suggest you complete the larger process of cloning curl and resubmit your stable changes then for review. - Sascha -- PHP CVS Mailing

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Sterling Hughes
> Sacha- > > I can appreciate that there is a limited potential need for PHP to support > chunked encoding >right-now<, however this is one step in a larger process > to support keep-alives and issuing multiple requests over a single > http/https session. I'd like to work on the issues raised b

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Sterling Hughes
Sascha Schumann wrote: HTTP/1.1 supports compressing, which is good to save bandwidth. As does HTTP/1.0. Refer to RFC 1945, section 3.5 Content Codings. As the code is only in HEAD, there is no real problem, because untill PHP 5.1 goes out, I'll test this extensively :) That is quit

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Sara Golemon
> This morning I've started testing this new feature and I've found some bugs: > * it is segfaulting on redirects > I'll just bet I know why too The way the http:// wrapper is built, any redirects would be prone to stacking multiple filters on top of each other. There's two things to address

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Sascha Schumann
HTTP/1.1 supports compressing, which is good to save bandwidth. As does HTTP/1.0. Refer to RFC 1945, section 3.5 Content Codings. As the code is only in HEAD, there is no real problem, because untill PHP 5.1 goes out, I'll test this extensively :) That is quite optimistic. If some

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Nuno Lopes
Well well well. A basic HTTP client such as PHP does not gain anything from supporting HTTP/1.1. Until the time this feature is proven to be stable (especially the dechunking is not always easy to do correctly and reliably in C), the code should live outside of the mainstream

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Sascha Schumann
Well well well. A basic HTTP client such as PHP does not gain anything from supporting HTTP/1.1. Until the time this feature is proven to be stable (especially the dechunking is not always easy to do correctly and reliably in C), the code should live outside of the mainstream

Re: [PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-08 Thread Nuno Lopes
Hello Sara :) This morning I've started testing this new feature and I've found some bugs: * it is segfaulting on redirects * the chunk decoding algorithm isn't working well. Just run a diff in thw output of both HTTP/1.0 and 1.1. The only thing I can help is with backtraces: * GDB: http://testes

[PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-09-07 Thread Sara Golemon
pollita Tue Sep 7 15:27:11 2004 EDT Modified files: /php-src/ext/standard http_fopen_wrapper.c /php-srcNEWS Log: Protocol version context option and chunked transfer encoding http://cvs.php.net/diff.php/php-src/ext/standard/http_fopen_wrapper.c?r1

[PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-03-19 Thread Ilia Alshanetsky
iliaa Fri Mar 19 10:42:41 2004 EDT Modified files: /php-srcNEWS /php-src/ext/standard http_fopen_wrapper.c Log: MFH: Fixed bug #27628 (Simplify the process of making a POST request via stream context). http://cvs.php.net/diff.php/php-src/NE

[PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2004-01-29 Thread Sara Golemon
pollita Thu Jan 29 19:24:19 2004 EDT Modified files: /php-src/ext/standard http_fopen_wrapper.c /php-srcNEWS Log: Fix potential bug in http:// proxy support. Some proxy servers require entire URI be sent in request string. Add context option

[PHP-CVS] cvs: php-src / NEWS /ext/standard http_fopen_wrapper.c

2003-12-02 Thread Sara Golemon
pollita Wed Dec 3 00:30:17 2003 EDT Modified files: /php-src/ext/standard http_fopen_wrapper.c /php-srcNEWS Log: Add http proxy support via context option. Index: php-src/ext/standard/http_fopen_wrapper.c diff -u php-src/ext/standard/http_fopen_w