You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by or...@apache.org on 2021/06/14 21:09:07 UTC

[qpid-broker-j] 01/03: NO-JIRA: Fix sporadic test failure

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

orudyy pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/qpid-broker-j.git

commit e55fda01b87eb093a326ec5d9a905cd214488b0f
Author: Alex Rudyy <or...@apache.org>
AuthorDate: Mon Jun 14 20:00:47 2021 +0100

    NO-JIRA: Fix sporadic test failure
---
 .../authentication/PreemptiveAuthenticationTest.java | 20 ++------------------
 1 file changed, 2 insertions(+), 18 deletions(-)

diff --git a/systests/qpid-systests-http-management/src/test/java/org/apache/qpid/tests/http/authentication/PreemptiveAuthenticationTest.java b/systests/qpid-systests-http-management/src/test/java/org/apache/qpid/tests/http/authentication/PreemptiveAuthenticationTest.java
index 14db61d..12e5483 100644
--- a/systests/qpid-systests-http-management/src/test/java/org/apache/qpid/tests/http/authentication/PreemptiveAuthenticationTest.java
+++ b/systests/qpid-systests-http-management/src/test/java/org/apache/qpid/tests/http/authentication/PreemptiveAuthenticationTest.java
@@ -32,6 +32,7 @@ import static org.hamcrest.Matchers.startsWith;
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.junit.Assert.fail;
 
+import java.io.IOException;
 import java.net.HttpURLConnection;
 import java.net.InetAddress;
 import java.net.SocketException;
@@ -146,27 +147,10 @@ public class PreemptiveAuthenticationTest extends HttpTestBase
             helper.getJson("broker/getUser", STRING_TYPE_REF, SC_OK);
             fail("Exception not thrown");
         }
-        catch (SSLHandshakeException e)
+        catch (IOException e)
         {
             // PASS
         }
-        catch (SSLException e)
-        {
-            // PASS
-            //
-            // TLS 1.3 seems has an issue with handshake (some sort of race):
-            // after the socket being closed on server side due to unknown certificate,
-            // the client is trying to flush output stream and gets "broken pipe".
-            //
-            // The issue seems related to https://bugs.openjdk.java.net/browse/JDK-8207009
-            // There is still a problem with handshake in 11.0.4 and above
-            //
-            // Ignoring this issue... due to low impact of the issue on client applications...
-        }
-        catch (SocketException e)
-        {
-            // TODO - defect - we are not always seeing the SSL handshake exception
-        }
     }
 
     @Test

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org