[java-identity-provider COMMIT] in /trunk/idp-core/src/main/java/net/shibboleth/idp/service: AbstractReloadableServic...

[noreply at shibboleth.net]

Author: scantor
Date: Sat Jul 19 01:20:20 2014
New Revision: 6314

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=6314&view=rev
Log:
Logging improvements.

Modified:
    trunk/idp-core/src/main/java/net/shibboleth/idp/service/AbstractReloadableService.java
    trunk/idp-core/src/main/java/net/shibboleth/idp/service/ReloadableSpringService.java

Modified: trunk/idp-core/src/main/java/net/shibboleth/idp/service/AbstractReloadableService.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-core/src/main/java/net/shibboleth/idp/service/AbstractReloadableService.java?rev=6314&r1=6313&r2=6314&view=diff
==============================================================================
--- trunk/idp-core/src/main/java/net/shibboleth/idp/service/AbstractReloadableService.java (original)
+++ trunk/idp-core/src/main/java/net/shibboleth/idp/service/AbstractReloadableService.java Sat Jul 19 01:20:20 2014
@@ -267,7 +267,7 @@
      * @throws ServiceException thrown if there is a problem reloading the service
      */
     protected void doReload() {
-
+        log.info("{} Reloading service configuration", getLogPrefix());
     }
 
     /**

Modified: trunk/idp-core/src/main/java/net/shibboleth/idp/service/ReloadableSpringService.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-core/src/main/java/net/shibboleth/idp/service/ReloadableSpringService.java?rev=6314&r1=6313&r2=6314&view=diff
==============================================================================
--- trunk/idp-core/src/main/java/net/shibboleth/idp/service/ReloadableSpringService.java (original)
+++ trunk/idp-core/src/main/java/net/shibboleth/idp/service/ReloadableSpringService.java Sat Jul 19 01:20:20 2014
@@ -258,8 +258,8 @@
     @Override protected void doReload() {
         super.doReload();
 
-        log.debug("Creating new ApplicationContext for service '{}'", getId());
-        log.debug("Reloading from {}", getServiceConfigurations());
+        log.debug("{} Creating new ApplicationContext for service '{}'", getLogPrefix(), getId());
+        log.debug("{} Reloading from {}", getLogPrefix(), getServiceConfigurations());
         final GenericApplicationContext appContext;
         try {
             appContext =
@@ -285,7 +285,7 @@
         //
         final T theComponent = service.getComponent();
 
-        log.debug("Testing that {} is a superclass of {}", theComponent.getClass(), theClaz);
+        log.debug("{} Testing that {} is a superclass of {}", getLogPrefix(), theComponent.getClass(), theClaz);
 
         if (!theClaz.isAssignableFrom(theComponent.getClass())) {
             //
@@ -315,7 +315,7 @@
             oldComponent.unloadComponent();
         }
         lastLoadFailed = false;
-        log.debug("Reload complete");
+        log.info("{} Reload complete", getLogPrefix());
     }
 
     /** {@inheritDoc} */