thrackle / express-zip

stream multiple files to the browser as a single zip, in pure node
MIT License
72 stars 18 forks source link

express 3 #2

Closed btmdave closed 9 years ago

btmdave commented 11 years ago

Is this suppose to be supported in 3.x? Seems to just continuously be "starting..." but doesn't ever complete.

thrackle commented 11 years ago

Hi Dave. Thanks for the report. I'll try to take a look into this. Of course, if you discover anything please let me know.

thrackle commented 11 years ago

The problem is with the zipstream dependency:

"Recursive process.nextTick detected. This will break in the next version of node. Please use setImmediate for recursive deferral."

Working on a fix...

btmdave commented 11 years ago

Ah very cool, thank you :)

thrackle commented 9 years ago

Fixed in 2.0.0 (just published) by PR #5.