Fixed missing http entity (json) logging

Marcus Linke · Marcus Linke · commit 973a2a0d4349 · 2014-09-13T01:19:26.000+02:00
diff --git a/src/main/java/com/github/dockerjava/core/SelectiveLoggingFilter.java b/src/main/java/com/github/dockerjava/core/SelectiveLoggingFilter.java
@@ -2,6 +2,7 @@
 
 import java.io.IOException;
 import java.util.Set;
+import java.util.logging.Logger;
 
 import javax.ws.rs.client.ClientRequestContext;
 import javax.ws.rs.core.HttpHeaders;
@@ -25,7 +26,11 @@ public class SelectiveLoggingFilter extends LoggingFilter {
             .add("application/tar")
             .build();
 
-    @Override
+    public SelectiveLoggingFilter(Logger logger, boolean b) {
+		super(logger, b);
+	}
+
+	@Override
     public void filter(ClientRequestContext context) throws IOException {
         // Unless the content type is in the list of those we want to ellide, then just have
         // our super-class handle things.
diff --git a/src/main/java/com/github/dockerjava/jaxrs/DockerCmdExecFactoryImpl.java b/src/main/java/com/github/dockerjava/jaxrs/DockerCmdExecFactoryImpl.java
@@ -1,6 +1,7 @@
 package com.github.dockerjava.jaxrs;
 
 import java.io.IOException;
+import java.util.logging.Logger;
 
 import javax.ws.rs.client.Client;
 import javax.ws.rs.client.ClientBuilder;
@@ -52,6 +53,9 @@ public class DockerCmdExecFactoryImpl implements DockerCmdExecFactory {
 
 	private WebTarget baseResource;
 	
+	private static final Logger LOGGER = Logger.getLogger(DockerCmdExecFactoryImpl.class.getName());
+    
+	
 	@Override
 	public void init(DockerClientConfig dockerClientConfig) {
 		Preconditions.checkNotNull(dockerClientConfig, "config was not specified");
@@ -62,9 +66,11 @@ public void init(DockerClientConfig dockerClientConfig) {
         clientConfig.register(JacksonJsonProvider.class);
 
         if (dockerClientConfig.isLoggingFilterEnabled()) {
-            clientConfig.register(SelectiveLoggingFilter.class);
+            clientConfig.register(new SelectiveLoggingFilter(LOGGER, true));
         }
 
+        //clientConfig.register(new LoggingFilter(LOGGER, true));
+        
         if (dockerClientConfig.getReadTimeout() != null) {
         	int readTimeout = dockerClientConfig.getReadTimeout();
         	clientConfig.property(ClientProperties.READ_TIMEOUT, readTimeout);
