Eclipse
/
jetty.project
mirror of https://github.com/jetty/jetty.project.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed #2585 

Signed-off-by: Greg Wilkins <gregw@webtide.com>
This commit is contained in:
Greg Wilkins 2018-05-29 15:48:53 +02:00
parent 4f20447a5a
commit 436c00b0af
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
jetty-http2/http2-hpack/src/main/java/org/eclipse/jetty/http2/hpack/HpackDecoder.java
View File

@ -246,8 +246,9 @@ public class HpackDecoder
if (indexed) if (indexed)
{ {
// add to dynamic table // add to dynamic table
if (_context.add(field)==null) _context.add(field);
throw new BadMessageException(HttpStatus.REQUEST_HEADER_FIELDS_TOO_LARGE_431,"Indexed field value too large"); // if (_context.add(field)==null)
// throw new BadMessageException(HttpStatus.REQUEST_HEADER_FIELDS_TOO_LARGE_431,"Indexed field value too large");
} }
} }

4
tests/test-http-client-transport/src/test/java/org/eclipse/jetty/http/client/HttpClientStreamTest.java
View File

@ -457,12 +457,12 @@ public class HttpClientStreamTest extends AbstractTest
public void testInputStreamResponseListenerFailedBeforeResponse() throws Exception public void testInputStreamResponseListenerFailedBeforeResponse() throws Exception
{ {
start(new EmptyServerHandler()); start(new EmptyServerHandler());
int port = connector instanceof ServerConnector?ServerConnector.class.cast( connector ).getLocalPort():1; String uri = newURI();
server.stop(); server.stop();
InputStreamResponseListener listener = new InputStreamResponseListener(); InputStreamResponseListener listener = new InputStreamResponseListener();
// Connect to the wrong port // Connect to the wrong port
client.newRequest("localhost",port) client.newRequest(uri)
.scheme(getScheme()) .scheme(getScheme())
.send(listener); .send(listener);
Result result = listener.await(5, TimeUnit.SECONDS); Result result = listener.await(5, TimeUnit.SECONDS);