Skip to content

Commit

Permalink
Try skipping certs
Browse files Browse the repository at this point in the history
  • Loading branch information
damccorm committed Dec 23, 2024
1 parent d737e5b commit 64d391e
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@
import software.amazon.awssdk.http.apache.ApacheHttpClient;
import software.amazon.awssdk.http.apache.ProxyConfiguration;
import software.amazon.awssdk.http.nio.netty.NettyNioAsyncHttpClient;
import software.amazon.awssdk.internal.http.NoneTlsKeyManagersProvider;
import software.amazon.awssdk.regions.Region;

/**
Expand Down Expand Up @@ -177,6 +178,9 @@ public <BuilderT extends AwsClientBuilder<BuilderT, ClientT>, ClientT> BuilderT
setOptional(httpConfig.maxConnections(), client::maxConnections);
}

// TODO - gate this behind a flag.
client.tlsKeyManagersProvider(NoneTlsKeyManagersProvider.getInstance());

// must use builder to make sure client is managed by the SDK
((SdkSyncClientBuilder<?, ?>) builder).httpClientBuilder(client);
} else if (builder instanceof SdkAsyncClientBuilder) {
Expand All @@ -199,6 +203,9 @@ public <BuilderT extends AwsClientBuilder<BuilderT, ClientT>, ClientT> BuilderT
httpConfig.socketTimeout(), client::writeTimeout); // fallback for writeTimeout
setOptional(httpConfig.writeTimeout(), client::writeTimeout);
setOptional(httpConfig.maxConnections(), client::maxConcurrency);

// TODO - gate this behind a flag.
client.tlsKeyManagersProvider(NoneTlsKeyManagersProvider.getInstance());
}

// must use builder to make sure client is managed by the SDK
Expand Down

0 comments on commit 64d391e

Please sign in to comment.