Skip to content

Commit

Permalink
Merge pull request #27 from Nosto/fix/first-line-length
Browse files Browse the repository at this point in the history
Bumped the first-line HTTP header length to 16384 bytes
  • Loading branch information
mridang committed Jun 30, 2017
2 parents d214271 + 678c24d commit 19c3f60
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion framework/src/play/server/PlayHandler.java
Original file line number Diff line number Diff line change
Expand Up @@ -729,7 +729,7 @@ public void exceptionCaught(ChannelHandlerContext ctx, ExceptionEvent e) throws
// Log this, we can't call serve500()
Throwable t = e.getCause();
if (t instanceof TooLongFrameException) {
Logger.error(t, "Request exceeds 8192 bytes");
Logger.error(t, "Request exceeds 16384 bytes");
}
e.getChannel().close();
} catch (Exception ex) {
Expand Down
2 changes: 1 addition & 1 deletion framework/src/play/server/PlayHttpRequestDecoder.java
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@
public class PlayHttpRequestDecoder extends HttpRequestDecoder {

public PlayHttpRequestDecoder() {
super(8192, 8192, 8192);
super(16384, 16384, 16384);
}
}

0 comments on commit 19c3f60

Please sign in to comment.