Skip to content

Added support for https destination, http-response and mikeal's request streams.#7

Merged
alexindigo merged 4 commits intoform-data:masterfrom
alexindigo:master
Aug 4, 2012
Merged

Added support for https destination, http-response and mikeal's request streams.#7
alexindigo merged 4 commits intoform-data:masterfrom
alexindigo:master

Conversation

@alexindigo
Copy link
Copy Markdown
Member

Use cases:

  • https destination: upload photos to api.500px.com
  • remote file streams (http-response, mikeal's request): Upload photos from Flickr to 500px.com or vice versa.

…st streams.

Use cases:

- https destination: upload photos to api.500px.com

- remote file streams (http-response, mikeal's request): Upload photos from Flickr to 500px.com or vice versa.
@alexindigo
Copy link
Copy Markdown
Member Author

And I made workaround for tests (incoming form parsing) since formidable is broken.

@felixge
Copy link
Copy Markdown
Contributor

felixge commented Jul 30, 2012

And I made workaround for tests (incoming form parsing) since formidable is broken.

What's broken in formidable?

I'm not using this module at the moment, so I'm happy to merge this patch (and also give you npm access to publish versions if you like). Two things so:

  • Could you add an example for your features to the Readme?
  • Your new test test-http-respone.js seems have a typo in the filename (missing s).

@alexindigo
Copy link
Copy Markdown
Member Author

It failed on form.parse(req); in original checkout,
and when I went to formidable github page it showed that travis test broken,
on the travis page it showed it's broken for node 0.4 and node 0.6,
so I realized you guys up to something. :)

And just added workaround for the tests here.
(Going to revert it back when formidable is back to normal).

Thanx, fixed the typo.

Take a look at Readme if it's ok, you you want something more elaborated.

I'm heavily relaying on this module in my project,
so I don't mind to publish updates to npm.

Thank you.

@felixge
Copy link
Copy Markdown
Contributor

felixge commented Aug 2, 2012

I added another comment, and also made a you a collaborator on the GitHub project (so you can merge this yourself if you're happy with this).

If you give me your npm user name, I'll also give you npm publish permissions.

@alexindigo
Copy link
Copy Markdown
Member Author

I fixed styling. Going to merge it.

My npm name: alexindigo.

Thanx.

alexindigo added a commit that referenced this pull request Aug 4, 2012
Added support for https destination, http-response and mikeal's request streams.
@alexindigo alexindigo merged commit 3832297 into form-data:master Aug 4, 2012
@felixge
Copy link
Copy Markdown
Contributor

felixge commented Aug 5, 2012

Thanks. Added you as an npm contributor as well. Feel free to publish versions.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants