[activemq] branch activemq-5.15.x updated: [AMQ-7303] Improve logging in case of security exception on transport connection

[jbonofre]

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

jbonofre pushed a commit to branch activemq-5.15.x
in repository https://gitbox.apache.org/repos/asf/activemq.git


The following commit(s) were added to refs/heads/activemq-5.15.x by this push:
     new bf3891e  [AMQ-7303] Improve logging in case of security exception on 
transport connection
bf3891e is described below

commit bf3891ed0edbc1ec74e9052df8cd0f9df2c7e579
Author: jbonofre <jbono...@apache.org>
AuthorDate: Wed Mar 4 18:57:02 2020 +0100

    [AMQ-7303] Improve logging in case of security exception on transport 
connection
    
    (cherry picked from commit 945cd6b7de20cbc0a092eaa2e6dff5dfacc9acc3)
---
 .../src/main/java/org/apache/activemq/broker/TransportConnection.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/activemq-broker/src/main/java/org/apache/activemq/broker/TransportConnection.java
 
b/activemq-broker/src/main/java/org/apache/activemq/broker/TransportConnection.java
index b1e1dce..d242ef7 100644
--- 
a/activemq-broker/src/main/java/org/apache/activemq/broker/TransportConnection.java
+++ 
b/activemq-broker/src/main/java/org/apache/activemq/broker/TransportConnection.java
@@ -852,7 +852,7 @@ public class TransportConnection implements Connection, 
Task, CommandVisitor {
                 brokerConnectionStates.remove(info.getConnectionId());
             }
             unregisterConnectionState(info.getConnectionId());
-            LOG.warn("Failed to add Connection id={}, clientId={} due to {}", 
info.getConnectionId(), clientId, e);
+            LOG.warn("Failed to add Connection id={}, clientId={}, clientIP={} 
due to {}", info.getConnectionId(), clientId, info.getClientIp(), 
e.getLocalizedMessage());
             //AMQ-6561 - stop for all exceptions on addConnection
             // close this down - in case the peer of this transport doesn't 
play nice
             delayedStop(2000, "Failed with SecurityException: " + 
e.getLocalizedMessage(), e);