You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Oct 5, 2022. It is now read-only.
The FabricDiscoveryAgent class (FQDN: io.fabric8.mq.fabric.discovery.FabricDiscoveryAgent) expects system properties for the URL and password of the ZK.
This is not a clean implementation, especially in a world of JNDI, Spring and properties files. It forces the user to pass properties to the runtime where their application runs to add these things, especially if the application runs inside some application server/JVM container where these things are not segregatable.
I then request that these properties be configurable from the "top", ie. from the ActiveMQConnectionFactory so that the ZK's URL and Password can be read from a proper configuration source.
Also, the default to "localhost:2181" (on the call() method) doesn't make much sense. Usually libraries connect where they're told and not to defaults buried deep inside the code.
The FabricDiscoveryAgent class (FQDN: io.fabric8.mq.fabric.discovery.FabricDiscoveryAgent) expects system properties for the URL and password of the ZK.
This is not a clean implementation, especially in a world of JNDI, Spring and properties files. It forces the user to pass properties to the runtime where their application runs to add these things, especially if the application runs inside some application server/JVM container where these things are not segregatable.
I then request that these properties be configurable from the "top", ie. from the ActiveMQConnectionFactory so that the ZK's URL and Password can be read from a proper configuration source.
Also, the default to "localhost:2181" (on the call() method) doesn't make much sense. Usually libraries connect where they're told and not to defaults buried deep inside the code.
master dependency: io.fabric8.mq:mq-fabric:1.2.0.redhat-621084
The text was updated successfully, but these errors were encountered: