[activemq-artemis] branch master updated: ARTEMIS-2803 fix semantic change

[clebertsuconic]

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

clebertsuconic pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git


The following commit(s) were added to refs/heads/master by this push:
     new 9d4ec41  ARTEMIS-2803 fix semantic change
     new 337e130  This closes #3184
9d4ec41 is described below

commit 9d4ec419358b3c462fd12a533e581df0f302c683
Author: Justin Bertram <jbert...@apache.org>
AuthorDate: Mon Jun 15 12:04:17 2020 -0500

    ARTEMIS-2803 fix semantic change
---
 .../activemq/artemis/core/server/cluster/ClusterController.java       | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/ClusterController.java
 
b/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/ClusterController.java
index 1805b0d..f6b32de 100644
--- 
a/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/ClusterController.java
+++ 
b/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/ClusterController.java
@@ -359,8 +359,8 @@ public class ClusterController implements ActiveMQComponent 
{
                   clusterConnection = 
server.getClusterManager().getDefaultConnection(null);
                }
 
-               //if there is no default cluster connection then just ignore 
the packet with a log message
-               if (clusterConnection == null) {
+               //if there is no default cluster connection and security is 
enabled then just ignore the packet with a log message
+               if (clusterConnection == null && 
server.getConfiguration().isSecurityEnabled()) {
                   
ActiveMQServerLogger.LOGGER.failedToFindClusterConnection(packet.toString());
                   return;
                }