You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ro...@apache.org on 2017/11/07 10:24:34 UTC
[sling-org-apache-sling-tracer] 03/10: SLING-6089 - Log Tracer does
not work with system having new HTTP whiteboard implementation
This is an automated email from the ASF dual-hosted git repository.
rombert pushed a commit to annotated tag org.apache.sling.tracer-1.0.2
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-tracer.git
commit 6813db4aad12dda6057b899c16d18288b49e90cc
Author: Chetan Mehrotra <ch...@apache.org>
AuthorDate: Mon Oct 3 11:41:30 2016 +0000
SLING-6089 - Log Tracer does not work with system having new HTTP whiteboard implementation
As mentioned by Robert using constant does not lead to adding package dependency on the constant class package
git-svn-id: https://svn.apache.org/repos/asf/sling/trunk/contrib/extensions/tracer@1763155 13f79535-47bb-0310-9956-ffa450edef68
---
pom.xml | 7 ++++++-
src/main/java/org/apache/sling/tracer/internal/LogTracer.java | 11 ++++-------
2 files changed, 10 insertions(+), 8 deletions(-)
diff --git a/pom.xml b/pom.xml
index 8c68597..8291063 100644
--- a/pom.xml
+++ b/pom.xml
@@ -137,7 +137,12 @@
<version>3.0.0</version>
<scope>provided</scope>
</dependency>
-
+ <dependency>
+ <groupId>org.osgi</groupId>
+ <artifactId>org.osgi.service.http.whiteboard</artifactId>
+ <version>1.0.0</version>
+ <scope>provided</scope>
+ </dependency>
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
diff --git a/src/main/java/org/apache/sling/tracer/internal/LogTracer.java b/src/main/java/org/apache/sling/tracer/internal/LogTracer.java
index c9ec0e6..714f02b 100644
--- a/src/main/java/org/apache/sling/tracer/internal/LogTracer.java
+++ b/src/main/java/org/apache/sling/tracer/internal/LogTracer.java
@@ -54,6 +54,7 @@ import org.apache.sling.commons.osgi.PropertiesUtil;
import org.osgi.framework.BundleContext;
import org.osgi.framework.Constants;
import org.osgi.framework.ServiceRegistration;
+import org.osgi.service.http.whiteboard.HttpWhiteboardConstants;
import org.slf4j.LoggerFactory;
import org.slf4j.Marker;
@@ -272,14 +273,10 @@ public class LogTracer {
Dictionary<String, Object> filterProps = new Hashtable<String, Object>();
filterProps.put("pattern", "/.*");
- //Do not use constant name to keep dependency as optional
- //filterProps.put(HttpWhiteboardConstants.HTTP_WHITEBOARD_FILTER_PATTERN, "/");
- //filterProps.put(HttpWhiteboardConstants.HTTP_WHITEBOARD_CONTEXT_SELECT,
- // "(" + HttpWhiteboardConstants.HTTP_WHITEBOARD_CONTEXT_NAME + "=*)");
+ filterProps.put(HttpWhiteboardConstants.HTTP_WHITEBOARD_FILTER_PATTERN, "/");
+ filterProps.put(HttpWhiteboardConstants.HTTP_WHITEBOARD_CONTEXT_SELECT,
+ "(" + HttpWhiteboardConstants.HTTP_WHITEBOARD_CONTEXT_NAME + "=*)");
- filterProps.put("osgi.http.whiteboard.filter.pattern", "/");
- filterProps.put("osgi.http.whiteboard.context.select",
- "(osgi.http.whiteboard.context.name=*)");
filterProps.put(Constants.SERVICE_DESCRIPTION, "Servlet Filter required for Log Tracer");
filterRegistration = context.registerService(Filter.class.getName(),
new TracerFilter(), filterProps);
--
To stop receiving notification emails like this one, please contact
"commits@sling.apache.org" <co...@sling.apache.org>.