Mailing-List: contact users-help@activemq.apache.org; run by ezmlm
Precedence: bulk
Reply-To: users@activemq.apache.org
Received-SPF: pass (athena.apache.org: local policy)
Message-ID: <50C9E1C6.6050308@jive-videos.net>
Date: Thu, 13 Dec 2012 14:10:14 +0000
From: Rob Coward <rob@jive-videos.net>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64;
 rv:17.0) Gecko/17.0 Thunderbird/17.0
MIME-Version: 1.0
To: users@activemq.apache.org
Subject: Re: Broker-to-broker Authentication with LDAP Broker Discovery
References: <50C5FAB3.9070504@jive-videos.net>
 <CAGeh-pH+C_0RTtwkGcxDLu6TLDCi39=WKAdy4Kz2+LDjHd0iww@mail.gmail.com>
In-Reply-To: 
 <CAGeh-pH+C_0RTtwkGcxDLu6TLDCi39=WKAdy4Kz2+LDjHd0iww@mail.gmail.com>
Content-Type: multipart/mixed;
 boundary="------------030509040706090703080106"

--------------030509040706090703080106
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Hi Dejan,
Many thanks for your reply. I decided to take a stab at the code myself 
and the attached patch not only seems to now allow me to authenticate 
the connection, but also solves the problem of the discovered connector 
not registering in JMX. I hope others might find it useful - is it worth 
trying to push it up into trunk for future releases ?

Regards,
Rob

On 11/12/2012 12:51, Dejan Bosanac wrote:
> Hi Rob,
>
> unfortunately ldap network connector is not used much and so there
> were no much development on it.
>
> Alternatively, you can check http://fuse.fusesource.org/mq/docs/mq-fabric.html
>
> which uses ZooKeeper for broker discovery.
>
>
> Regards
> --
> Dejan Bosanac
> ----------------------
> Red Hat, Inc.
> FuseSource is now part of Red Hat
> dbosanac@redhat.com
> Twitter: @dejanb
> Blog: http://sensatic.net
> ActiveMQ in Action: http://www.manning.com/snyder/
>
>
> On Mon, Dec 10, 2012 at 4:07 PM, Rob Coward <rob@jive-videos.net> wrote:
>> http://activemq.apache.org/ldap-broker-discovery-mechanism.html


--------------030509040706090703080106
Content-Type: text/plain; charset=windows-1252;
 name="LdapNetworkConnector.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
 filename="LdapNetworkConnector.patch"

diff -aurN activemq-5.7.0-orig/activemq-core/src/main/java/org/apache/activemq/network/LdapNetworkConnector.java activemq-5.7.0/activemq-core/src/main/java/org/apache/activemq/network/LdapNetworkConnector.java
--- activemq-5.7.0-orig/activemq-core/src/main/java/org/apache/activemq/network/LdapNetworkConnector.java	2012-12-11 22:49:45.000000000 +0000
+++ activemq-5.7.0/activemq-core/src/main/java/org/apache/activemq/network/LdapNetworkConnector.java	2012-12-13 10:41:07.000000000 +0000
@@ -295,7 +295,11 @@
         // want to map/manage these differently in the future
         // boolean useJMX = getBrokerService().isUseJmx();
         // getBrokerService().setUseJmx(false);
-        NetworkConnector connector = getBrokerService().addNetworkConnector(connectorURI);
+	NetworkConnector discoveryconnector = new DiscoveryNetworkConnector(connectorURI);
+	discoveryconnector.setName(uuid);
+	discoveryconnector.setUserName(getUserName());
+	discoveryconnector.setPassword(this.password);
+        NetworkConnector connector = getBrokerService().addNetworkConnector(discoveryconnector);
         // getBrokerService().setUseJmx(useJMX);
 
         // Propagate standard connector properties that may have been set via XML

--------------030509040706090703080106--