[#7814] Look at using a stream when downloading zip files
Summary Look at using a stream when downloading zip files
Queue Horde Framework Packages
Queue Version Git master
Type Enhancement
State Resolved
Priority 1. Low
Requester mrubinsk (at) horde (dot) org
Created 2008-12-29 (3797 days ago)
Updated 2011-04-12 (2963 days ago)
Resolved 2011-04-12 (2963 days ago)
Milestone 2
Patch No

2011-04-12 08:58:09 Michael Slusarz Comment #7
State ⇒ Resolved
Reply to this comment
This has been completed.
2010-07-02 19:08:19 Michael Slusarz State ⇒ Stalled
2009-07-13 23:41:38 Michael Slusarz Comment #6
State ⇒ Feedback
Reply to this comment
I've added stream support to the zip compress method.  There is a 
limitation however - all data passed in must be converted at some 
point to an in-memory stream before passing through gzdeflate() - I 
can't get zlib stream filtering to work, although I have a hunch it is 
directly related to http://bugs.php.net/?id=48725
2009-03-18 17:48:29 Chuck Hagenbuch Comment #5 Reply to this comment
I guess the question for me is how widely available the pecl 
extensions are. Except for bugs, using our PHP code vs. PEAR or 
someone else's PHP code is kind of a wash to me - control vs. bugs vs. 
flexibility, etc.
2009-03-18 14:53:04 Michael Slusarz Comment #4 Reply to this comment
(A bit broader goal:) I personally would like to see us get out of 
compression data generation all together - meaning removing PHP code 
and instead exclusively using the other solutions available (PEAR/PECL 
libraries).  pear has evolved to the point that requiring a pear/pecl 
package is not difficult/prohibitive that we need a entirely local, 
PHP-based fallback IMHO.
2009-03-18 13:55:13 Michael Rubinsky Comment #3
Version ⇒ Git master
Queue ⇒ Horde Framework Packages
Reply to this comment
Moving to Framework queue for Horde 4 since this should be implemented 
in Horde_Compress
2009-02-13 02:46:46 Michael Rubinsky Milestone ⇒ 2
2009-01-06 02:42:32 Chuck Hagenbuch Comment #2
New Attachment: zipstream-php-0.1.1.zip Download
Reply to this comment
Code for inspiration or adaptation, from 
2008-12-29 21:13:03 Michael Rubinsky Comment #1
Type ⇒ Enhancement
State ⇒ Accepted
Priority ⇒ 1. Low
Summary ⇒ Look at using a stream when downloading zip files
Queue ⇒ Ansel
Milestone ⇒
Patch ⇒ No
Reply to this comment
from chuck:

and probably changing our zip generation code to stream, instead of 
just hitting the heck out of server memory

Saved Queries