HTTP::Tiny needs two modules for https support

[jkeenan]

When running sync-with-cpan on a machine for the first time, internal
subroutine wget() was silently failing. After adding debugging code and
running through the debugger, the HTTP::Tiny mirror() method call
displayed this error message:

IO::Socket::SSL 1.42 must be installed for https support
Net::SSLeay 1.49 must be installed for https support

[Grinnz]

They just need to be installed, not loaded, so does this help other than documenting the requirement?

[xenu]

It makes it fail loudly and early.

[Grinnz]

Ah I missed the problem that the response is not checked for any errors.

[Grinnz]

You may wish to consider error handling like this: https://metacpan.org/release/HTTP-Simple/source/lib/HTTP/Simple.pm#L81-83

[wwlwpd]

I see the installation of IO::Socket::SSL fail surprisingly often due to missing or not found openssl libraries. I don't think just requiring those modules is going to cut it for this, unless you want to make HTTP::Tiny dependent of the https only use case. (not all people need it).

[Grinnz]

This PR is not related to the dependencies of HTTP::Tiny itself but just the core developer process using it in this case.

[jkeenan]

Merged in commit 34c2009. Closing.