Fix ALPN whitespace

Fixes amphp/http-client#255. Noticed while comparing the TLS handshake with Chrome's TLS handshake.
amphp · Feb 27, 2020 · 020b652 · 020b652
1 parent 7083250
commit 020b652
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/ClientTlsContext.php b/src/ClientTlsContext.php
@@ -430,7 +430,7 @@ public function toStreamContextArray(): array
         }
 
         if (!empty($this->alpnProtocols)) {
-            $options['alpn_protocols'] = \implode(', ', $this->alpnProtocols);
+            $options['alpn_protocols'] = \implode(',', $this->alpnProtocols);
         }
 
         return ['ssl' => $options];

diff --git a/src/ServerTlsContext.php b/src/ServerTlsContext.php
@@ -413,7 +413,7 @@ public function toStreamContextArray(): array
         ];
 
         if (!empty($this->alpnProtocols)) {
-            $options['alpn_protocols'] = \implode(', ', $this->alpnProtocols);
+            $options['alpn_protocols'] = \implode(',', $this->alpnProtocols);
         }
 
         if ($this->defaultCertificate !== null) {