From 63456311b84d82cdb6209ee41988f8f26ba7d551 Mon Sep 17 00:00:00 2001 From: mturk Date: Sun, 26 Nov 2006 09:29:15 +0000 Subject: [PATCH] Compress outgoing response headers according to the AJP13 protocol specification. git-svn-id: https://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk@479317 13f79535-47bb-0310-9956-ffa450edef68 --- java/org/apache/coyote/ajp/AjpAprProcessor.java | 10 ++++-- java/org/apache/coyote/ajp/AjpProcessor.java | 8 ++++- java/org/apache/coyote/ajp/Constants.java | 44 +++++++++++++++++++++++++ 3 files changed, 59 insertions(+), 3 deletions(-) diff --git a/java/org/apache/coyote/ajp/AjpAprProcessor.java b/java/org/apache/coyote/ajp/AjpAprProcessor.java index dda1cef4c..3ce901d51 100644 --- a/java/org/apache/coyote/ajp/AjpAprProcessor.java +++ b/java/org/apache/coyote/ajp/AjpAprProcessor.java @@ -960,9 +960,15 @@ public class AjpAprProcessor implements ActionHook { // Other headers int numHeaders = headers.size(); responseHeaderMessage.appendInt(numHeaders); - for (int i = 0; i < numHeaders; i++) { + for (int i = 0; i < numHeaders; i++) { MessageBytes hN = headers.getName(i); - responseHeaderMessage.appendBytes(hN); + int hC = Constants.getResponseAjpIndex(hN.toString()); + if (hC > 0) { + responseHeaderMessage.appendInt(hC); + } + else { + responseHeaderMessage.appendBytes(hN); + } MessageBytes hV=headers.getValue(i); responseHeaderMessage.appendBytes(hV); } diff --git a/java/org/apache/coyote/ajp/AjpProcessor.java b/java/org/apache/coyote/ajp/AjpProcessor.java index b06d216cb..2161f2e4e 100644 --- a/java/org/apache/coyote/ajp/AjpProcessor.java +++ b/java/org/apache/coyote/ajp/AjpProcessor.java @@ -929,7 +929,13 @@ public class AjpProcessor implements ActionHook { responseHeaderMessage.appendInt(numHeaders); for (int i = 0; i < numHeaders; i++) { MessageBytes hN = headers.getName(i); - responseHeaderMessage.appendBytes(hN); + int hC = Constants.getResponseAjpIndex(hN.toString()); + if (hC > 0) { + responseHeaderMessage.appendInt(hC); + } + else { + responseHeaderMessage.appendBytes(hN); + } MessageBytes hV=headers.getValue(i); responseHeaderMessage.appendBytes(hV); } diff --git a/java/org/apache/coyote/ajp/Constants.java b/java/org/apache/coyote/ajp/Constants.java index 6074b495e..21bef8dd7 100644 --- a/java/org/apache/coyote/ajp/Constants.java +++ b/java/org/apache/coyote/ajp/Constants.java @@ -17,6 +17,9 @@ package org.apache.coyote.ajp; +import java.lang.IndexOutOfBoundsException; +import java.util.Hashtable; +import java.util.Locale; import org.apache.tomcat.util.buf.ByteChunk; @@ -67,6 +70,7 @@ public final class Constants { public static final int SC_RESP_SERVLET_ENGINE = 0xA009; public static final int SC_RESP_STATUS = 0xA00A; public static final int SC_RESP_WWW_AUTHENTICATE = 0xA00B; + public static final int SC_RESP_AJP13_MAX = 11; // Integer codes for common (optional) request attribute names public static final byte SC_A_CONTEXT = 1; // XXX Unused @@ -179,7 +183,47 @@ public final class Constants { "user-agent" }; + // Translates integer codes to response header names + public static final String []responseTransArray = { + "content-type", + "content-language", + "content-length", + "date", + "last-modified", + "location", + "set-cookie", + "set-cookie2", + "servlet-engine", + "status", + "www-authenticate" + }; + private static final Hashtable responseTransHash = + new Hashtable(20); + + static { + try { + int i; + for (i = 0; i < SC_RESP_AJP13_MAX; i++) { + responseTransHash.put(responseTransArray[i], + new Integer(0xA001 + i)); + } + } + catch (Exception e) { + // Do nothing + } + } + + public static final int getResponseAjpIndex(String header) + { + Integer i = responseTransHash.get(header.toLowerCase(Locale.US)); + if (i == null) + return 0; + else + return i.intValue(); + } + + /** * CRLF. */ -- 2.11.0