ID: 29045
Comment by: infos at firmalarsirketler dot com
Reported By: fcartegnie at nordnet dot fr
Status: Suspended
Bug Type: Documentation problem
Operating System: *
PHP Version: 4.3.7
Assigned To: pollita
New Comment:
thank you for this nice site
Previous Comments:
------------------------------------------------------------------------
[2004-07-08 11:22:48] [EMAIL PROTECTED]
You cannot drop STREAM_MUST_SEEK because the handler uses
seek. In case of http or ftp or what ever forward streams a temporary
file is created automatically so it still works as expected.
------------------------------------------------------------------------
[2004-07-07 22:17:42] [EMAIL PROTECTED]
Wez needs to chime in on this...
I stated that it wouldn't work for http (with or without the patch)
because I thought we had disabled shrinking the chunk_size to 1 while
reading into the HTTP headers (for performance reasons). Looking back
at the code I see that the temporary shift into small chuck sizes *IS*
enabled when the STREAM_WILL_CAST flag is set (which it is for the zlib
fopen wrapper).
Maybe it was the conditional set based on the option flag that was the
compromise... in which case I'd say: Go ahead and use
fopen('compress.zlib://http://blah...',... but the patch provided for
gzopen() seems completely unnecessary.
------------------------------------------------------------------------
[2004-07-07 16:16:46] [EMAIL PROTECTED]
compress.zlib://http://host.file.gz should work fine in php 4, not?
include "compress.zlib://file.php.gz"; works too ;-)
------------------------------------------------------------------------
[2004-07-07 16:12:31] [EMAIL PROTECTED]
compress.zlib://host/path isn't actually valid.
What is valid however (as of PHP5) is:
$fp = fopen('http://host/path/file.gz', 'r');
/* Maybe a small fread() here to consume gzip headers, I don't recall
if they're sent with deflated content */
stream_filter_append($fp, 'zlib.inflate');
Note: For PHP 5.0 you need PECL package zlib_filter, it'll be rolled
into the core in PHP 5.1 .
The patch supplied above needs further review as I see a big potential
for zlib stream errors with the read buffer blocks that come in with
the headers.
------------------------------------------------------------------------
[2004-07-07 14:39:51] fcartegnie at nordnet dot fr
compress.zlib://http:// seems to work
Maybe we could add some deprecation/pointer info in the documentation
of gzopen()
Changing for documentation problem
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://bugs.php.net/29045
--
Edit this bug report at http://bugs.php.net/?id=29045&edit=1
