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

[cl...@apache.org]

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 <jb...@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;
                }