Skip to content

Commit 0c4f879

Browse files
Issue #10337 - fixes to SizeLimitHandler
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
1 parent d4d8832 commit 0c4f879

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

jetty-server/src/main/java/org/eclipse/jetty/server/handler/SizeLimitHandler.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,6 @@
3131
import org.eclipse.jetty.server.HttpOutput;
3232
import org.eclipse.jetty.server.Request;
3333
import org.eclipse.jetty.util.Callback;
34-
import org.eclipse.jetty.util.log.Log;
35-
import org.eclipse.jetty.util.log.Logger;
3634

3735
/**
3836
* <p>A handler that can limit the size of message bodies in requests and responses.</p>
@@ -46,8 +44,6 @@
4644
*/
4745
public class SizeLimitHandler extends HandlerWrapper
4846
{
49-
private static final Logger LOG = Log.getLogger(SizeLimitHandler.class);
50-
5147
private final long _requestLimit;
5248
private final long _responseLimit;
5349

@@ -90,7 +86,7 @@ public void handle(String target, Request baseRequest, HttpServletRequest reques
9086
baseRequest.getHttpInput().addInterceptor(limit);
9187
}
9288

93-
if (_responseLimit > 0)
89+
if (_responseLimit >= 0)
9490
{
9591
httpOutput.setInterceptor(limit);
9692
response = new LimitResponse(response);
@@ -132,7 +128,7 @@ public HttpInput.Content readFrom(HttpInput.Content content)
132128
if (content.hasContent())
133129
{
134130
_read += content.remaining();
135-
checkResponseLimit(_read);
131+
checkRequestLimit(_read);
136132
}
137133
return content;
138134
}

0 commit comments

Comments
 (0)