urlToOpts should include protocol to support simply getting https

[faeldt]

urlToOpts is not including protocol so

get('https://myurl.com',
        function(err, getResponse) {

did not work since it was still trying http.

[feross]

Thanks for the PR!

I knew I should have added an https test along with the rest of the tests. :) Rectified in 816341f

Released as 1.1.0.

[faeldt]

Np. I wasn't sure how to make a test for https so I appreciate the lesson in the linked commit. :)

[feross]

Yeah, it's slightly more annoying which is why I originally skipped it ;)
On Mon, Jan 5, 2015 at 3:41 AM Kristian Faeldt notifications@github.com
wrote:

Np. I wasn't sure how to make a test for https so I appreciate the lesson
in the linked commit. :)

—
Reply to this email directly or view it on GitHub
#1 (comment).