From 2d6cfdf35f661e7385c8e20399d84c84187e487d Mon Sep 17 00:00:00 2001 From: Idel Pivnitskiy Date: Fri, 12 Mar 2021 00:49:56 -0800 Subject: [PATCH] Allow client security configuration without trust material (#1430) Motivation: Trust material is optional for the client-side security configuration and we should allow using the default JDK truststore. Modifications: - If users of deprecated `ClientSecurityConfig` did not specify a trust material, use default `ClientSslConfigBuilder` ctor; Result: Users of deprecated `ClientSecurityConfig` can create a security config with default JDK truststore. --- .../transport/netty/internal/ClientSecurityConfig.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/servicetalk-transport-netty-internal/src/main/java/io/servicetalk/transport/netty/internal/ClientSecurityConfig.java b/servicetalk-transport-netty-internal/src/main/java/io/servicetalk/transport/netty/internal/ClientSecurityConfig.java index 2fa3d7b1ef..f4f869a5e6 100644 --- a/servicetalk-transport-netty-internal/src/main/java/io/servicetalk/transport/netty/internal/ClientSecurityConfig.java +++ b/servicetalk-transport-netty-internal/src/main/java/io/servicetalk/transport/netty/internal/ClientSecurityConfig.java @@ -252,7 +252,7 @@ public ClientSslConfig asSslConfig() { } else if (trustCertChainSupplier != null) { builder = new ClientSslConfigBuilder(trustCertChainSupplier); } else { - throw new IllegalStateException("required trust material not set"); + builder = new ClientSslConfigBuilder(); } if (hostnameVerificationAlgorithm == null) {