diff --git a/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/ClientLogFilter.java b/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/ClientLogFilter.java deleted file mode 100644 index 9f21bb7..0000000 --- a/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/ClientLogFilter.java +++ /dev/null @@ -1,39 +0,0 @@ -package no.obos.util.servicebuilder.client; - -import lombok.extern.slf4j.Slf4j; - -import javax.annotation.Priority; -import javax.ws.rs.Priorities; -import javax.ws.rs.client.ClientRequestContext; -import javax.ws.rs.client.ClientRequestFilter; -import javax.ws.rs.client.ClientResponseContext; -import javax.ws.rs.client.ClientResponseFilter; -import java.io.IOException; - -@Priority(Priorities.USER) -@Slf4j -public class ClientLogFilter implements ClientRequestFilter, ClientResponseFilter { - public static final String PROPERTYNAME = "ClientLogFilter.startTime"; - - @Override - public void filter(ClientRequestContext requestContext) throws IOException { - requestContext.setProperty(PROPERTYNAME, System.nanoTime()); - log.info(getCallSignature(requestContext)); - } - - @Override - public void filter(ClientRequestContext requestContext, ClientResponseContext responseContext) throws IOException { - Long totalMillis = null; - - Long startNanos = (Long) requestContext.getProperty(PROPERTYNAME); - if (startNanos != null) { - long totalNanos = System.nanoTime() - startNanos; - totalMillis = totalNanos / 1_000_000; - } - log.info(getCallSignature(requestContext) + " response: " + responseContext.getStatus() + ", millis: " + totalMillis); - } - - private static String getCallSignature(ClientRequestContext requestContext) { - return requestContext.getMethod() + " " + requestContext.getUri().toString(); - } -} diff --git a/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/StubGenerator.java b/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/StubGenerator.java index 03aa89a..101e1eb 100644 --- a/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/StubGenerator.java +++ b/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/StubGenerator.java @@ -54,9 +54,6 @@ public T generateClient(Class resource) { } webTarget.register(RequestIdClientFilter.class); webTarget.register(ClientNameFilter.class); - if (logging) { - webTarget.register(ClientLogFilter.class); - } return WebResourceFactory.newResource(resource, webTarget, false, headerArg, cookies, new Form()); } diff --git a/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/TargetGenerator.java b/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/TargetGenerator.java index 200f92d..15764a8 100644 --- a/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/TargetGenerator.java +++ b/servicebuilder-core/src/main/java/no/obos/util/servicebuilder/client/TargetGenerator.java @@ -47,9 +47,6 @@ public WebTarget generate() { } target.register(RequestIdClientFilter.class); target.register(ClientNameFilter.class); - if (logging) { - target.register(ClientLogFilter.class); - } return target; }