Works with 2.4.x patch: http://people.apache.org/~ylavic/httpd-2.4.x-ap_http_filter_chunked.patch

-----Original Message----- From: Yann Ylavic
Sent: Sunday, June 14, 2015 3:27 AM Newsgroups: gmane.comp.apache.devel
To: dev@httpd.apache.org
Subject: Re: ********* Re: [VOTE] Release Apache httpd 2.4.14 as GA

On Sun, Jun 14, 2015 at 2:43 AM, Jeff Trawick <traw...@gmail.com> wrote:
On Sat, Jun 13, 2015 at 8:25 PM, Yann Ylavic <ylavic....@gmail.com> wrote:

On Sun, Jun 14, 2015 at 2:18 AM, Jeff Trawick <traw...@gmail.com> wrote:
> On Sat, Jun 13, 2015 at 7:52 PM, Yann Ylavic <ylavic....@gmail.com>
> wrote:
>>
>> On Sat, Jun 13, 2015 at 8:49 PM, Jeff Trawick <traw...@gmail.com>
>> wrote:
>> >
>> > Those that know the original patch would presumably have a better >> > way
>> > to
>> > code this.
>> >
>> > I recall that httpd 1.3 generally added whitespace after the chunk
>> > size
>> > as
>> > an optimization (pre-allocating space in BUFF for the largest
>> > possible
>> > chunk
>> > size I guess). Is it appropriate to allow whitespace both before >> > and
>> > after?
>> > The patch above only allows it after.
>>
>> Committed in r1685345, by allowing both space and tab, before and
>> after the chunk-size.
>> Thinking more about it, I see no reason to allow spaces before
>> (especially if there is no compatibility issue), we probably should
>> revert that part.
>> Opinions?
>
>
> 2.2 doesn't accept space or tab before the chunk size, so we don't need
> to
> accept that.

OK, fixed in r1685347.


Works for me :)

Backports to 2.4.x and 2.2.x proposed.
Thanks!

Reply via email to