This is an automated email from the ASF dual-hosted git repository.

fanningpj pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-pekko-grpc.git


The following commit(s) were added to refs/heads/main by this push:
     new b9a533be Fixes https://github.com/playframework/play-grpc/issues/468 
(#197)
b9a533be is described below

commit b9a533be65c7e83cc44e41adfb0fcded601fb704
Author: Sergey Morgunov <[email protected]>
AuthorDate: Mon Dec 25 15:38:48 2023 +0300

    Fixes https://github.com/playframework/play-grpc/issues/468 (#197)
---
 .../scala/org/apache/pekko/grpc/internal/NettyClientUtils.scala  | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git 
a/runtime/src/main/scala/org/apache/pekko/grpc/internal/NettyClientUtils.scala 
b/runtime/src/main/scala/org/apache/pekko/grpc/internal/NettyClientUtils.scala
index 7d27799a..9ce0a6bc 100644
--- 
a/runtime/src/main/scala/org/apache/pekko/grpc/internal/NettyClientUtils.scala
+++ 
b/runtime/src/main/scala/org/apache/pekko/grpc/internal/NettyClientUtils.scala
@@ -184,6 +184,7 @@ object NettyClientUtils {
   private def createNettySslContext(javaSslContext: SSLContext): SslContext = {
     import io.grpc.netty.shaded.io.netty.handler.ssl.{
       ApplicationProtocolConfig,
+      ApplicationProtocolNames,
       ClientAuth,
       IdentityCipherSuiteFilter,
       JdkSslContext
@@ -195,8 +196,12 @@ object NettyClientUtils {
       /* boolean isClient */ true,
       /* Iterable<String> ciphers */ null, // use JDK defaults (null is 
accepted as indicated in constructor Javadoc)
       IdentityCipherSuiteFilter.INSTANCE,
-      /* ApplicationProtocolConfig apn */ ApplicationProtocolConfig.DISABLED, 
// use JDK default (null would also be acceptable, DISABLED config will select 
the NONE protocol and thus the JdkDefaultApplicationProtocolNegotiator)
-      ClientAuth.NONE, // server-only option, which is ignored as 
isClient=true (as indicated in constructor Javadoc)
+      new ApplicationProtocolConfig(
+        ApplicationProtocolConfig.Protocol.ALPN,
+        ApplicationProtocolConfig.SelectorFailureBehavior.NO_ADVERTISE,
+        ApplicationProtocolConfig.SelectedListenerFailureBehavior.ACCEPT,
+        ApplicationProtocolNames.HTTP_2),
+      ClientAuth.OPTIONAL, // server-only option, which is ignored as 
isClient=true (as indicated in constructor Javadoc)
       /* String[] protocols */ null, // use JDK defaults (null is accepted as 
indicated in constructor Javadoc)
       /* boolean startTls */ false)
   }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to