summaryrefslogtreecommitdiffstats
path: root/jdisc_http_service/src/main/java/com/yahoo/jdisc
diff options
context:
space:
mode:
authorBjørn Christian Seime <bjorncs@verizonmedia.com>2021-01-21 15:25:45 +0100
committerBjørn Christian Seime <bjorncs@verizonmedia.com>2021-01-21 16:54:09 +0100
commitbe7847731521fb516269b65905029c1639d4aec8 (patch)
treee044d0c13d3e7fc66f92cf1fb9a96a6f3a8c4203 /jdisc_http_service/src/main/java/com/yahoo/jdisc
parent99dff1d3c48ac9a481f25831c877bc51109848eb (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.java26
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);
- }
-
- }
}