Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Added support for uploading files to buffer instead of filesystem #267

Open
wants to merge 11 commits into
from

Conversation

Projects
None yet
2 participants

Elzair commented Jan 2, 2014

A while ago, I encountered a problem uploading images >1MB to a Nodejitsu instance. I assume that cloud hosting providers like Nodejitsu do not always support a large filesystem for node.js instances. Since I needed to upload the file to the server before pushing it to an image hosting service, I modified node-formidable to support uploading files to Buffers instead of to the filesystem.

Owner

felixge commented Jan 3, 2014

Hi @Elzair - thanks for this contribution. I have to admit that I'm a little hesitant when it comes to merging this, as the desired functionality can be provided in userland as well (by defining your own form.onPart = function(part) { handler).

Do you see any compelling reasons why this should be part of formidable itself?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment