replace deprecated http.createClient() usage

[mscdex]

No description provided.

[mreinstein]

thanks Brian. Curious, why http.Agent ? The docs indicate http.request is the sanctioned replacement:

https://nodejs.org/dist/latest-v6.x/docs/api/http.html#http_http_createclient_port_host

[mscdex]

I was being a bit more strict in the replacement since the http.Client constructor created its own http.Agent instance. http.request() uses a global Agent instance by default that is shared by all http.request() calls.

Either way it's up to you what you want to use, if you feel http.request() without creating an explicit http.Agent instance is good enough in this particular case, then feel free to use that instead of this PR. At the very least you can view this PR as a heads up that the deprecated http client API may be removed in the near future (nodejs/node#8104).

[mreinstein]

ok sounds quite reasonable. Thanks again.