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

Deque from the acceptEncodingQueue even if it's not gonna be encoded

Browse Source
This commit is contained in:
Trustin Lee 2012-06-27 10:27:43 +09:00
parent 59a1095c23
commit fe6b71c1f0
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/main/java/org/jboss/netty/handler/codec/http/HttpContentEncoder.java
View File

@ -92,6 +92,11 @@ public abstract class HttpContentEncoder extends SimpleChannelHandler {
encoder = null;
String acceptEncoding = acceptEncodingQueue.poll();
if (acceptEncoding == null) {
throw new IllegalStateException("cannot send more responses than requests");
}
String contentEncoding = m.getHeader(HttpHeaders.Names.CONTENT_ENCODING);
if (contentEncoding != null &&
!HttpHeaders.Values.IDENTITY.equalsIgnoreCase(contentEncoding)) {
@ -99,11 +104,6 @@ public abstract class HttpContentEncoder extends SimpleChannelHandler {
ctx.sendDownstream(e);
} else {
// Determine the content encoding.
String acceptEncoding = acceptEncodingQueue.poll();
if (acceptEncoding == null) {
throw new IllegalStateException("cannot send more responses than requests");
}
boolean hasContent = m.isChunked() || m.getContent().readable();
if (hasContent && (encoder = newContentEncoder(m, acceptEncoding)) != null) {
// Encode the content and remove or replace the existing headers