aboutsummaryrefslogtreecommitdiffstats
path: root/container-core
diff options
context:
space:
mode:
authorBjørn Christian Seime <bjorncs@verizonmedia.com>2021-03-25 17:13:58 +0100
committerBjørn Christian Seime <bjorncs@verizonmedia.com>2021-03-25 17:14:52 +0100
commitfff3ece9d0a627523eba0cec5ff32ed6b945aee5 (patch)
tree08e98d766a5d0d5d37a772265fc4beb57fe963d5 /container-core
parentb7b15f6ab97d12f675e180a0ea215500086d08ad (diff)
Add test verifying request size in request log
Diffstat (limited to 'container-core')
-rw-r--r--container-core/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java15
1 files changed, 15 insertions, 0 deletions
diff --git a/container-core/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java b/container-core/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java
index cb7e3ef347f..b0107a29f2c 100644
--- a/container-core/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java
+++ b/container-core/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java
@@ -867,6 +867,21 @@ public class HttpServerTest {
Assertions.assertThat(logEntry.sslPeerNotAfter()).hasValue(Instant.EPOCH.plus(100_000, ChronoUnit.DAYS));
}
+ @Test
+ public void requireThatRequestIsTrackedInAccessLog() throws IOException {
+ InMemoryRequestLog requestLogMock = new InMemoryRequestLog();
+ TestDriver driver = TestDrivers.newConfiguredInstance(
+ new EchoRequestHandler(),
+ new ServerConfig.Builder(),
+ new ConnectorConfig.Builder(),
+ binder -> binder.bind(RequestLog.class).toInstance(requestLogMock));
+ driver.client().newPost("/status.html").setContent("abcdef").execute().expectStatusCode(is(OK));
+ assertThat(driver.close(), is(true));
+ RequestLogEntry entry = requestLogMock.entries().get(0);
+ Assertions.assertThat(entry.statusCode()).hasValue(200);
+ Assertions.assertThat(entry.requestSize()).hasValue(6);
+ }
+
private ContentResponse sendJettyClientRequest(TestDriver testDriver, Path certificateFile, Object tag)
throws Exception {
HttpClient client = createJettyHttpClient(certificateFile);