William A. Rowe, Jr. wrote:
Jeff Trawick wrote:
On 9/8/05, William A. Rowe, Jr. <[EMAIL PROTECTED]> wrote:
I've provided the following fork of that codebase, to try to repair the
damage from the vetoed 171205 commit, in a piece by piece analysis of
what's changed and why.
any particular reason we lost the sendunchangedcl setting?
+ if (...
+ (r->input_filters == r->proto_input_filters
+ ...
+ || apr_table_get(r->subprocess_env,
"proxy-sendunchangedcl"))) {
+ rb_method = RB_STREAM_CL;
+ }
But there's a second justification; because the new patch pre-reads up
to some fixed bytes of input, and handle the vast majority of POST
bodies in memory with a trusted CL, the necessity is somewhat lessened.
And I missed the third point; if the *protocol* input filter chain has
not been modified (no input request filters injected) then we continue
to presume that the CL is unchanged. So really the only code that
triggers spool is the situation where we have injected an input filter,
and we cannot (will not) chunk the request body, and the body is larger
than the read-ahead buffer. So this is most definately the exception,
not the rule, and the 'advantages' of some proxy-sendunchangedcl envvar
just seem not to justify the probability of misconfiguration and the
unhopefull task of diagnosing the actual symptoms.
Bill
