summaryrefslogtreecommitdiffstats
path: root/jdisc_http_service/src/test
diff options
context:
space:
mode:
authorArnstein Ressem <aressem@gmail.com>2017-09-20 20:11:28 +0200
committerGitHub <noreply@github.com>2017-09-20 20:11:28 +0200
commit281e135347407443957d3ad420f69552d63efe4a (patch)
tree3a58e535bc993b5b1d3614423ddb0c57976ea2ac /jdisc_http_service/src/test
parente6120f4dba393a473647d7b242fcfc0a40974b25 (diff)
Revert "Use Jetty v9.4.7"
Diffstat (limited to 'jdisc_http_service/src/test')
-rw-r--r--jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java8
1 files changed, 3 insertions, 5 deletions
diff --git a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java
index 55dec64e967..7ed13decbf6 100644
--- a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java
+++ b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java
@@ -371,7 +371,6 @@ public class HttpServerTest {
}
// Header with no value is disallowed by https://tools.ietf.org/html/rfc7230#section-3.2
- // Details in https://github.com/eclipse/jetty.project/issues/1116
@Test
public void requireThatHeaderWithNullValueIsOmitted() throws Exception {
final TestDriver driver = TestDrivers.newInstance(new EchoWithHeaderRequestHandler("X-Foo", null));
@@ -381,14 +380,13 @@ public class HttpServerTest {
assertThat(driver.close(), is(true));
}
- // Header with empty value is allowed by https://tools.ietf.org/html/rfc7230#section-3.2
- // Details in https://github.com/eclipse/jetty.project/issues/1116
+ // Header with no value is disallowed by https://tools.ietf.org/html/rfc7230#section-3.2
@Test
- public void requireThatHeaderWithEmptyValueIsAllowed() throws Exception {
+ public void requireThatHeaderWithEmptyValueIsOmitted() throws Exception {
final TestDriver driver = TestDrivers.newInstance(new EchoWithHeaderRequestHandler("X-Foo", ""));
driver.client().get("/status.html")
.expectStatusCode(is(OK))
- .expectHeader("X-Foo", is(""));
+ .expectNoHeader("X-Foo");
assertThat(driver.close(), is(true));
}