socketry / multipart-post

Adds multipart POST capability to net/http
MIT License
293 stars 72 forks source link

Fix overriding content-type header for FileParts #43

Closed ehutzelman closed 7 years ago

ehutzelman commented 8 years ago

It looks like the headers parameter in build_head is no longer used, and all headers are merged into opts hash. This PR removes the extraneous param and looks for Content-Type header in opts instead.

ioquatix commented 7 years ago

I'm just working on fixing the tests, once that's done I'll take a look at this PR.

ioquatix commented 7 years ago

I know this has been a long time, but do you mind rebasing on the current head?

ehutzelman commented 7 years ago

Rebased, had to remind myself what this was about :)

ioquatix commented 7 years ago

Thanks!