Avoid setting ALPN if not supported

Fixes #283.
amphp · Dec 13, 2020 · 1c1f383 · 1c1f383
1 parent 00c610a
commit 1c1f383
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/Connection/DefaultConnectionFactory.php b/src/Connection/DefaultConnectionFactory.php
@@ -91,13 +91,12 @@ public function create(
                 }
 
                 $tlsContext = ($connectContext->getTlsContext() ?? new ClientTlsContext(''))
-                    ->withApplicationLayerProtocols($protocols)
                     ->withPeerCapturing();
 
-                if ($protocols === ['http/1.1']) {
-                    // If we only have HTTP/1.1 available, don't set application layer protocols.
-                    // There are misbehaving sites like n11.com, see https://github.com/amphp/http-client/issues/255
-                    $tlsContext = $tlsContext->withApplicationLayerProtocols([]);
+                // If we only have HTTP/1.1 available, don't set application layer protocols.
+                // There are misbehaving sites like n11.com, see https://github.com/amphp/http-client/issues/255
+                if ($protocols !== ['http/1.1'] && Socket\hasTlsAlpnSupport()) {
+                    $tlsContext = $tlsContext->withApplicationLayerProtocols($protocols);
                 }
 
                 if ($tlsContext->getPeerName() === '') {