QPID-1394 : Added test to validate that Provider is registered after connection is created and logging to highlight if we are unable to register our client connections. We do not need to unregister our clients as the same providers are used for all new connections.

git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk@707691 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 8 insertions, 2 deletions

diff --git a/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java b/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java
index 9f48ac96a3..d2bf979d0a 100644
--- a/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java
+++ b/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java
@@ -39,8 +39,14 @@ public class DynamicSaslRegistrar
         Map<String, Class> factories = registerSaslClientFactories();
         if (factories.size() > 0)
         {
-            Security.addProvider(new JCAProvider(factories));
-            _logger.debug("Dynamic SASL provider added as a security provider");
+            if (Security.addProvider(new JCAProvider(factories)) == -1)
+            {
+                _logger.error("Unable to load custom SASL providers. Qpid custom SASL authenticators unavailable.");                
+            }
+            else
+            {
+                _logger.debug("Dynamic SASL provider added as a security provider");
+            }
         }
     }