You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by bd...@apache.org on 2009/09/14 14:21:45 UTC

svn commit: r814606 - in /sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal: SlingServletResolver.java resource/ServletResourceProvider.java

Author: bdelacretaz
Date: Mon Sep 14 12:21:45 2009
New Revision: 814606

URL: http://svn.apache.org/viewvc?rev=814606&view=rev
Log:
Improved logging of servlet registrations

Modified:
    sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java
    sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProvider.java

Modified: sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java?rev=814606&r1=814605&r2=814606&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java (original)
+++ sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java Mon Sep 14 12:21:45 2009
@@ -732,6 +732,7 @@
         ServiceRegistration reg = context.getBundleContext().registerService(
             ResourceProvider.SERVICE_NAME, provider, params);
 
+        log.info("Registered {}", provider.toString());
         servletsByReference.put(reference, reg);
 
         return true;

Modified: sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProvider.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProvider.java?rev=814606&r1=814605&r2=814606&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProvider.java (original)
+++ sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProvider.java Mon Sep 14 12:21:45 2009
@@ -18,6 +18,7 @@
  */
 package org.apache.sling.servlets.resolver.internal.resource;
 
+import java.util.Arrays;
 import java.util.Iterator;
 import java.util.Set;
 
@@ -71,5 +72,12 @@
     public String[] getSerlvetPaths() {
         return resourcePaths.toArray(new String[resourcePaths.size()]);
     }
-
+    
+    /** Return suitable info for logging */
+    @Override
+    public String toString() {
+    	return getClass().getSimpleName() 
+    	+ ": servlet=" + servlet.getClass().getName() 
+    	+ ", paths=" + Arrays.asList(resourcePaths);
+    }
 }