From c89eba71410e23319b008c5e0c75decc161adb0e Mon Sep 17 00:00:00 2001
From: Scott Battaglia <scott.battaglia@gmail.com>
Date: Sun, 30 Nov 2014 17:09:09 -0500
Subject: [PATCH] Updated try/catch loop to log when a class is not found.

---
 .../cas/client/configuration/ConfigurationStrategyName.java | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/cas-client-core/src/main/java/org/jasig/cas/client/configuration/ConfigurationStrategyName.java b/cas-client-core/src/main/java/org/jasig/cas/client/configuration/ConfigurationStrategyName.java
index ea089fe..bc257f2 100644
--- a/cas-client-core/src/main/java/org/jasig/cas/client/configuration/ConfigurationStrategyName.java
+++ b/cas-client-core/src/main/java/org/jasig/cas/client/configuration/ConfigurationStrategyName.java
@@ -1,6 +1,8 @@
 package org.jasig.cas.client.configuration;
 
 import org.jasig.cas.client.util.CommonUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * Enumeration to map simple names to the underlying classes so that deployers can reference the simple name in the
@@ -13,6 +15,8 @@ public enum ConfigurationStrategyName {
 
     DEFAULT(LegacyConfigurationStrategyImpl.class), JNDI(JndiConfigurationStrategyImpl.class), WEB_XML(WebXmlConfigurationStrategyImpl.class);
 
+    private static final Logger LOGGER = LoggerFactory.getLogger(ConfigurationStrategyName.class);
+
     private final Class<? extends ConfigurationStrategy> configurationStrategyClass;
 
     private ConfigurationStrategyName(final Class<? extends ConfigurationStrategy> configurationStrategyClass) {
@@ -43,7 +47,7 @@ public enum ConfigurationStrategyName {
                 return (Class<? extends ConfigurationStrategy>) clazz;
             }
         }   catch (final ClassNotFoundException e) {
-            // nothing we can do here
+            LOGGER.error("Unable to locate strategy {} by name or class name.  Using default strategy instead.", value, e);
         }
 
         return DEFAULT.configurationStrategyClass;