In http.js, do not end request for keep-alive connections

[mgorczyca]

The request.end(data) line was preempting maintaining a persistent connection, even when Connection: 'keep-alive' was specified. My proposal is to handle the data via the body option, and only end the request when keep-alive is not specified.

[herom]

Thanks a lot for your contribution @mgorczyca - we appreciate it a lot 👍

The TravisCI Build is failing due to failing tests in node v0.12 and io.js - in order to test it with your local repo it would be necessary to use nvm (at least I guess that there is no other/better solution) and install node v0.12 as well as io.js and run npm test in these environments too.

[mgorczyca]

The build for v0.8.28 is failing with the following:

Error: Cannot find module 'readable-stream/duplex'

The npm test completes successfully in my nvm environment for the same node.js version. I'd appreciate any suggestion or solution.

[herom]

Sorry for the late reply - I restarted the Travis CI tests and everything is passing (sometimes Travis CI seems to have some hickups while running tests).

Please, in order to get your PR merged in quickly, follow the steps lined out in our Guideline on Submitting a Pull Request. The points which are left to merge your PR are the following:

• Please, squash your commits into a single one. If you don't know how to do this there are links to the excellent GitHub help on this topic in our Guidelines.
• Please, please, please * do not close a pull request due to a request to rebase* (instead, read through our short r Guidelines and find all necessary information & help in order to squash/rebase your commits 👍

[mgorczyca]

Thanks for the helpful information and links!

[herom]

Thanks a lot 👍