You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ri...@apache.org on 2008/10/24 19:30:15 UTC
svn commit: r707691 -
/incubator/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/security/DynamicSaslRegistrar.java
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");
+ }
}
}