We should have the DefaultHostnameVerifier do this internally, rather than loading the class.  Also, if we don't have the Sun (or OpenJDK) provider then there's going to be a number of very large issues with configuration in general with the TLS featureset, so that needs looking at.

I think NonFatal is appropriate in this case because it only treats the following errors as fatal:

case _: VirtualMachineError | _: ThreadDeath | _: InterruptedException | _: LinkageError | _: ControlThrowable => false
     
https://github.com/scala/scala/blob/master/src/library/scala/util/control/NonFatal.scala 


.  Also, I think NonFatal IS appropriate here