Bug 34771 - Unable to set maxHttpHeaderSize
Summary: Unable to set maxHttpHeaderSize
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 4
Classification: Unclassified
Component: Connector:Coyote HTTP/1.1 (show other bugs)
Version: 4.1.31
Hardware: All All
: P2 normal (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-06 10:28 UTC by Robin Spainhour
Modified: 2005-09-20 16:20 UTC (History)
1 user (show)



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Robin Spainhour 2005-05-06 10:28:54 UTC
The value I set in maxHttpHeaderSize attribute of my connector is not picked up
on start up. Instead the default 4 KB setting is used.

The problem was introduced in 4.1.30 release when the connector code from 5.0
was back ported into 4.1 releases. 
(The code containing the error is still present at least up to release 5.0.30) 

The fix seems to be as simple as to just add setters and getter for
maxHttpHeaderSize property in org.apache.coyote.tomcat4.CoyoteConnector.

Further, there are a number of other properties that cannot be set using the
tomcat4 CoyoteConnecter. As I realize development does not focus on release 4.1
anymore, would it be possible to remove the "final" declaration of the
org.apache.coyote.tomcat4.CoyoteConnector?
Until we have a chance to upgrade to a more recent release, this would make it
easier for us to patch this class in a more controlled way without having to
bother you guys.
Comment 1 Mark Thomas 2005-05-23 00:25:56 UTC
I'm not going to remove the final declaration but if you attach a patch for the
additional attributes (diff -u format please) to this bug report I will take a
look at it.

You will need to bear in mind that any future 4.1.x release will most likely use
the 5.0.x branch of jakarta-tomcat-connectors.
Comment 2 Mark Thomas 2005-07-30 18:53:38 UTC
For the record, TC4 now builds using HEAD from the connectors and the next
release of TC4 (whenever that might be) will use the same connectors as the most
recent stable 5.5.x release.
Comment 3 Mark Thomas 2005-09-21 00:20:26 UTC
This is fixed in CVS and will be included in the next 4.1.x release (for which
no date has yet been set).