andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fixed issue: NETTY-107 - HttpMessageDecoder can not handle the content with no 'Content-Length' header.

Browse Source
This commit is contained in:
Trustin Lee 2009-02-12 07:32:53 +00:00
parent 2d682dc2a3
commit 431151b8e5
3 changed files with 17 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/java/org/jboss/netty/handler/codec/http/HttpMessageDecoder.java
View File

@ -103,10 +103,13 @@ public abstract class HttpMessageDecoder extends ReplayingDecoder<HttpMessageDec
if (!mergeChunks) {
return message;
}
} else if (message.getContentLength(-1) == 0) {
} else {
int contentLength = message.getContentLength(-1);
if (contentLength == 0 || contentLength == -1 && isDecodingRequest()) {
content = ChannelBuffers.EMPTY_BUFFER;
return reset();
}
}
//we return null here, this forces decode to be called again where we will decode the content
return null;
}
@ -246,6 +249,7 @@ public abstract class HttpMessageDecoder extends ReplayingDecoder<HttpMessageDec
checkpoint(nextState);
}
protected abstract boolean isDecodingRequest();
protected abstract void readInitial(ChannelBuffer buffer) throws Exception;
private int getChunkSize(String hex) {

5
src/main/java/org/jboss/netty/handler/codec/http/HttpRequestDecoder.java
View File

@ -49,4 +49,9 @@ public class HttpRequestDecoder extends HttpMessageDecoder {
HttpVersion.valueOf(split[2]), HttpMethod.valueOf(split[0]), split[1]);
checkpoint(State.READ_HEADER);
}
@Override
protected boolean isDecodingRequest() {
return true;
}
}

5
src/main/java/org/jboss/netty/handler/codec/http/HttpResponseDecoder.java
View File

@ -48,4 +48,9 @@ public class HttpResponseDecoder extends HttpMessageDecoder {
message = new DefaultHttpResponse(HttpVersion.valueOf(split[0]), new HttpResponseStatus(Integer.valueOf(split[1]), split[2]));
checkpoint(State.READ_HEADER);
}
@Override
protected boolean isDecodingRequest() {
return false;
}
}