diff options
author | Bjørn Christian Seime <bjorncs@vespa.ai> | 2023-11-17 10:48:11 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@vespa.ai> | 2023-11-17 10:48:11 +0100 |
commit | 50c6735bca827a49aa1b2e552d5e9f45903c2d4b (patch) | |
tree | 10126d2e339b9a00fa0cac3891a6feb1fe0dad79 /container-core/src/test/java/com/yahoo/jdisc/http/server | |
parent | 91692155c1f63492ef8d57b35105a5ade375a6ba (diff) |
Fail immediately when `Content-Length` is present
Diffstat (limited to 'container-core/src/test/java/com/yahoo/jdisc/http/server')
-rw-r--r-- | container-core/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java | 14 |
1 files changed, 14 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 6e218a6ab66..adb35db8ebf 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 @@ -160,6 +160,20 @@ public class HttpServerTest { } @Test + void requireThatTooLargePayloadFailsWith413() throws Exception { + final JettyTestDriver driver = JettyTestDriver.newConfiguredInstance( + new EchoRequestHandler(), + new ServerConfig.Builder(), + new ConnectorConfig.Builder() + .maxContentSize(100)); + driver.client().newPost("/status.html") + .setBinaryContent(new byte[200]) + .execute() + .expectStatusCode(is(REQUEST_TOO_LONG)); + assertTrue(driver.close()); + } + + @Test void requireThatMultipleHostHeadersReturns400() throws Exception { var metricConsumer = new MetricConsumerMock(); JettyTestDriver driver = JettyTestDriver.newConfiguredInstance( |