Skip to content

Commit

Permalink
Allow client security configuration without trust material (#1430)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
idelpivnitskiy authored Mar 12, 2021
1 parent dd91cb6 commit 2d6cfdf
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -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) {
Expand Down

0 comments on commit 2d6cfdf

Please sign in to comment.