diff options
author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2021-01-21 15:25:45 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2021-01-21 16:54:09 +0100 |
commit | be7847731521fb516269b65905029c1639d4aec8 (patch) | |
tree | e044d0c13d3e7fc66f92cf1fb9a96a6f3a8c4203 /jdisc_http_service/src/main/java/com/yahoo/jdisc | |
parent | 99dff1d3c48ac9a481f25831c877bc51109848eb (diff) |
Remove all unused fields from AccessLogEntry
Diffstat (limited to 'jdisc_http_service/src/main/java/com/yahoo/jdisc')
-rw-r--r-- | jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/AccessLoggingRequestHandler.java | 26 |
1 files changed, 1 insertions, 25 deletions
diff --git a/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/AccessLoggingRequestHandler.java b/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/AccessLoggingRequestHandler.java index e30d50ecdbf..842ab75a312 100644 --- a/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/AccessLoggingRequestHandler.java +++ b/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/AccessLoggingRequestHandler.java @@ -4,7 +4,6 @@ package com.yahoo.jdisc.http.server.jetty; import com.google.common.base.Preconditions; import com.yahoo.container.logging.AccessLogEntry; import com.yahoo.jdisc.Request; -import com.yahoo.jdisc.Response; import com.yahoo.jdisc.handler.AbstractRequestHandler; import com.yahoo.jdisc.handler.ContentChannel; import com.yahoo.jdisc.handler.RequestHandler; @@ -23,7 +22,6 @@ import java.util.Optional; * Does not otherwise interfere with the request processing of the delegate request handler. * * @author bakksjo - * $Id$ */ public class AccessLoggingRequestHandler extends AbstractRequestHandler { public static final String CONTEXT_KEY_ACCESS_LOG_ENTRY @@ -54,30 +52,8 @@ public class AccessLoggingRequestHandler extends AbstractRequestHandler { Preconditions.checkArgument(request instanceof HttpRequest, "Expected HttpRequest, got " + request); final HttpRequest httpRequest = (HttpRequest) request; httpRequest.context().put(CONTEXT_KEY_ACCESS_LOG_ENTRY, accessLogEntry); - final ResponseHandler accessLoggingResponseHandler = new AccessLoggingResponseHandler(httpRequest, handler, accessLogEntry); - final ContentChannel requestContentChannel = delegate.handleRequest(request, accessLoggingResponseHandler); - return requestContentChannel; + return delegate.handleRequest(request, handler); } - private static class AccessLoggingResponseHandler implements ResponseHandler { - private final HttpRequest request; - private final ResponseHandler delegateHandler; - private final AccessLogEntry accessLogEntry; - - public AccessLoggingResponseHandler( - HttpRequest request, final ResponseHandler delegateHandler, - final AccessLogEntry accessLogEntry) { - this.request = request; - this.delegateHandler = delegateHandler; - this.accessLogEntry = accessLogEntry; - } - - @Override - public ContentChannel handleResponse(Response response) { - accessLogEntry.setUserPrincipal(request.getUserPrincipal()); - return delegateHandler.handleResponse(response); - } - - } } |