From 86a21d661bdd3ecd39032b724e9ba5a2fe2d6db5 Mon Sep 17 00:00:00 2001 From: Ismael Juma Date: Tue, 20 Jul 2021 06:17:51 -0700 Subject: [PATCH] MINOR: Fix testTlsDefaults failure due to TLS 1.0/1.1 being disabled The latest JDKs no longer support TLS 1.0/1.1 causing the test to fail. We have already fixed this in trunk and 3.0, so this is for 2.8 and older branches. Trunk commits are 530224e4fe and 12aa595e17. --- .../kafka/common/network/SslTransportLayerTest.java | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/clients/src/test/java/org/apache/kafka/common/network/SslTransportLayerTest.java b/clients/src/test/java/org/apache/kafka/common/network/SslTransportLayerTest.java index b177a01690c1a..17a8d796a7f11 100644 --- a/clients/src/test/java/org/apache/kafka/common/network/SslTransportLayerTest.java +++ b/clients/src/test/java/org/apache/kafka/common/network/SslTransportLayerTest.java @@ -591,7 +591,7 @@ public void testInvalidKeyPassword(Args args) throws Exception { } /** - * Tests that connection success with the default TLS version. + * Tests that connection succeeds with the default TLS version. */ @ParameterizedTest @ArgumentsSource(SslTransportLayerArgumentsProvider.class) @@ -611,12 +611,6 @@ public void testTlsDefaults(Args args) throws Exception { NetworkTestUtils.checkClientConnection(selector, "0", 10, 100); server.verifyAuthenticationMetrics(1, 0); selector.close(); - - checkAuthenticationFailed(args, "1", "TLSv1.1"); - server.verifyAuthenticationMetrics(1, 1); - - checkAuthenticationFailed(args, "2", "TLSv1"); - server.verifyAuthenticationMetrics(1, 2); } /** Checks connection failed using the specified {@code tlsVersion}. */