You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@activemq.apache.org by ra...@apache.org on 2009/07/29 07:24:24 UTC

svn commit: r798785 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java

Author: rajdavies
Date: Wed Jul 29 05:24:24 2009
New Revision: 798785

URL: http://svn.apache.org/viewvc?rev=798785&view=rev
Log:
Fix for https://issues.apache.org/activemq/browse/AMQ-2335

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java?rev=798785&r1=798784&r2=798785&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java Wed Jul 29 05:24:24 2009
@@ -77,8 +77,12 @@
         super.addConnection(context, info);
 
         ActiveMQTopic topic = AdvisorySupport.getConnectionAdvisoryTopic();
-        fireAdvisory(context, topic, info);
-        connections.put(info.getConnectionId(), info);
+        //do not distribute usernames or passwords in advisory
+        ConnectionInfo copy = info.copy();
+        copy.setUserName("");
+        copy.setPassword("");
+        fireAdvisory(context, topic, copy);
+        connections.put(copy.getConnectionId(), copy);
     }
 
     public Subscription addConsumer(ConnectionContext context, ConsumerInfo info) throws Exception {