[jboss-cvs] JBoss Messaging SVN: r7666 - trunk/src/main/org/jboss/messaging/core/remoting/server/impl.

[jboss-cvs-commits at lists.jboss.org]

Author: jmesnil
Date: 2009-08-05 08:48:50 -0400 (Wed, 05 Aug 2009)
New Revision: 7666

Modified:
   trunk/src/main/org/jboss/messaging/core/remoting/server/impl/RemotingServiceImpl.java
Log:
JBMESSAGING-1699: Server fails with message "Did not receive initial ping for connection"

* display the connection's remote address in the warning logs & exception

Modified: trunk/src/main/org/jboss/messaging/core/remoting/server/impl/RemotingServiceImpl.java
===================================================================
--- trunk/src/main/org/jboss/messaging/core/remoting/server/impl/RemotingServiceImpl.java	2009-08-05 12:44:13 UTC (rev 7665)
+++ trunk/src/main/org/jboss/messaging/core/remoting/server/impl/RemotingServiceImpl.java	2009-08-05 12:48:50 UTC (rev 7666)
@@ -443,7 +443,7 @@
          if (!gotInitialPing)
          {
             // Never received initial ping
-            log.warn("Did not receive initial ping for connection, it will be closed");
+            log.warn("Did not receive initial ping from " + conn.getRemoteAddress() + ", connection will be closed");
 
             closeConnection(conn);
 
@@ -466,7 +466,7 @@
          removeConnection(conn.getID());
 
          MessagingException me = new MessagingException(MessagingException.CONNECTION_TIMEDOUT,
-                                                        "Did not receive ping from client. It is likely a client has exited or crashed without " + "closing its connection, or the network between the server and client has failed. The connection will now be closed.");
+                                                        "Did not receive ping from " + conn.getRemoteAddress() + ". It is likely the client has exited or crashed without " + "closing its connection, or the network between the server and client has failed. The connection will now be closed.");
 
          conn.fail(me);
       }