Don't set peer domain and port in SSLEngine when HttpsEndpointIdentificationAlgorithm is disabled, close AsyncHttpClient#1535

Motivation:

Some broken TLS implementations crash on SSL handshakes that have host
and port. Browers handle this by retrying without those parameters, but
that’s definitively not something we’ll implement.

Disabling HttpsEndpointIdentificationAlgorithm should suffice to be
able to connect to such broken server.

Modifications:

When HttpsEndpointIdentificationAlgorithm is disabled (that disables
SNI and hostname verification), we want to not set peer domain and port
on the SSLEngine so those are not sent in the SSL handshake and make
broken servers crash.

Result:

It’s now possible to connect to broken servers that don’t support
hostname and port in SSL handshake.

Author: slandelle

client/src/main/java/org/asynchttpclient/netty/ssl/DefaultSslEngineFactory.java

@@ -60,8 +60,10 @@ private SslContext buildSslContext(AsyncHttpClientConfig config) throws SSLExcep
 
   @Override
   public SSLEngine newSslEngine(AsyncHttpClientConfig config, String peerHost, int peerPort) {
-    // FIXME should be using ctx allocator
-    SSLEngine sslEngine = sslContext.newEngine(ByteBufAllocator.DEFAULT, domain(peerHost), peerPort);
+    SSLEngine sslEngine =
+      config.isDisableHttpsEndpointIdentificationAlgorithm() ?
+        sslContext.newEngine(ByteBufAllocator.DEFAULT) :
+        sslContext.newEngine(ByteBufAllocator.DEFAULT, domain(peerHost), peerPort);
     configureSslEngine(sslEngine, config);
     return sslEngine;
   }