svn commit: r707691 - /incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java

[ritchiem]

Author: ritchiem
Date: Fri Oct 24 10:30:15 2008
New Revision: 707691

URL: http://svn.apache.org/viewvc?rev=707691&view=rev
Log:
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.

Modified:
    
incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java

Modified: 
incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java
URL: 
http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java?rev=707691&r1=707690&r2=707691&view=diff
==============================================================================
--- 
incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java
 (original)
+++ 
incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java
 Fri Oct 24 10:30:15 2008
@@ -39,8 +39,14 @@
         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");
+            }
         }
     }