Copy raw path and query from request to access log entry

1 files changed, 18 insertions, 0 deletions

diff --git a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/AccessLogRequestLogTest.java b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/AccessLogRequestLogTest.java
index cf08346d4fb..cdcd2d76883 100644
--- a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/AccessLogRequestLogTest.java
+++ b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/AccessLogRequestLogTest.java
@@ -7,6 +7,8 @@ import org.testng.annotations.Test;
 
 import javax.servlet.http.HttpServletRequest;
 
+import java.util.Optional;
+
 import static org.hamcrest.CoreMatchers.is;
 import static org.hamcrest.CoreMatchers.not;
 import static org.hamcrest.CoreMatchers.nullValue;
@@ -72,4 +74,20 @@ public class AccessLogRequestLogTest {
         assertThat(accessLogEntry.getURI().toString(), is("/search/?q=%25%252"));
     }
 
+    @Test
+    public void raw_path_and_query_are_set_from_request() {
+        HttpServletRequest httpServletRequest = mock(HttpServletRequest.class);
+        String rawPath = "//search/";
+        when(httpServletRequest.getRequestURI()).thenReturn(rawPath);
+        String rawQuery = "q=%%2";
+        when(httpServletRequest.getQueryString()).thenReturn(rawQuery);
+
+        AccessLogEntry accessLogEntry = new AccessLogEntry();
+        AccessLogRequestLog.populateAccessLogEntryFromHttpServletRequest(httpServletRequest, accessLogEntry);
+        assertThat(accessLogEntry.getRawPath(), is(rawPath));
+        Optional<String> actualRawQuery = accessLogEntry.getRawQuery();
+        assertThat(actualRawQuery.isPresent(), is(true));
+        assertThat(actualRawQuery.get(), is(rawQuery));
+    }
+
 }