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

Posted to commits@activemq.apache.org by ce...@apache.org on 2012/11/30 14:42:48 UTC

svn commit: r1415631 - /activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java

Author: ceposta
Date: Fri Nov 30 13:42:48 2012
New Revision: 1415631

URL: http://svn.apache.org/viewvc?rev=1415631&view=rev
Log:
https://issues.apache.org/jira/browse/AMQ-4198 Include username in ConnectionInfo for ActiveMQ.Advisory.Connection messages

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

Modified: activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java?rev=1415631&r1=1415630&r2=1415631&view=diff
==============================================================================
--- activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java (original)
+++ activemq/trunk/activemq-broker/src/main/java/org/apache/activemq/advisory/AdvisoryBroker.java Fri Nov 30 13:42:48 2012
@@ -79,9 +79,8 @@ public class AdvisoryBroker extends Brok
         super.addConnection(context, info);
 
         ActiveMQTopic topic = AdvisorySupport.getConnectionAdvisoryTopic();
-        //do not distribute usernames or passwords in advisory
+        // do not distribute passwords in advisory messages. usernames okay
         ConnectionInfo copy = info.copy();
-        copy.setUserName("");
         copy.setPassword("");
         fireAdvisory(context, topic, copy);
         connections.put(copy.getConnectionId(), copy);