You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by kw...@apache.org on 2015/01/28 23:13:26 UTC

svn commit: r1655485 - in /qpid/trunk/qpid/java: common/src/main/java/org/apache/qpid/transport/Connection.java systests/src/test/java/org/apache/qpid/client/ssl/SSLTest.java

Author: kwall
Date: Wed Jan 28 22:13:26 2015
New Revision: 1655485

URL: http://svn.apache.org/r1655485
Log:
QPID-6336: Add extra logging to help understand occasional SSLTest failure showing on slow CI box

Modified:
    qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java
    qpid/trunk/qpid/java/systests/src/test/java/org/apache/qpid/client/ssl/SSLTest.java

Modified: qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java?rev=1655485&r1=1655484&r2=1655485&view=diff
==============================================================================
--- qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java (original)
+++ qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java Wed Jan 28 22:13:26 2015
@@ -537,6 +537,11 @@ public class Connection extends Connecti
         connectionLost.set(true);
         synchronized (lock)
         {
+            if(log.isDebugEnabled())
+            {
+                log.debug("exception: %s state : %s", e.getMessage(), state);
+            }
+
             switch (state)
             {
             case OPENING:

Modified: qpid/trunk/qpid/java/systests/src/test/java/org/apache/qpid/client/ssl/SSLTest.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/systests/src/test/java/org/apache/qpid/client/ssl/SSLTest.java?rev=1655485&r1=1655484&r2=1655485&view=diff
==============================================================================
--- qpid/trunk/qpid/java/systests/src/test/java/org/apache/qpid/client/ssl/SSLTest.java (original)
+++ qpid/trunk/qpid/java/systests/src/test/java/org/apache/qpid/client/ssl/SSLTest.java Wed Jan 28 22:13:26 2015
@@ -112,7 +112,9 @@ public class SSLTest extends QpidBrokerT
             }
             catch (JMSException e)
             {
-                assertTrue("Unexpected exception message", e.getMessage().contains("Unrecognized SSL message, plaintext connection?"));
+                // PASS
+                assertTrue("Unexpected exception message : " + e.getMessage(),
+                           e.getMessage().contains("Unrecognized SSL message, plaintext connection?"));
             }
         }
     }



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