On Fri, Apr 20, 2018 at 11:50 AM, Eric Covener <cove...@gmail.com> wrote: > > Not sure about change but re: comment > > Are those comparable? The new proxy parm ResponseFieldSize is like > LimitRequestFieldSize not LimitRequestBody?
Yes indeed, I made a confusion here (comment restored in r1829643). Regarding the code change itself (!proxyreq => proxyreq != PROXYREQ_RESPONSE), I think this is the right thing to do because otherwise we don't limit proxyied *request* bodies. Actually I had this patch locally for a while, just noticing it now by adding the ap_rgetline() stuff (that I need to "release" ASAP :) . Will re-check if this is the case. This change is possibly not backportable to 2.4 though.