Re: Is 2.2.x broken?

2016-12-06 Thread Eric Covener 
The fix is the thing I just added to STATUS today

On Tue, Dec 6, 2016 at 1:07 PM, Jacob Champion  wrote:
> I'm trying to test a backport to the 2.2.x tip, but it looks like r1758671
> might have busted our request parsing. Every test we have fails; I think
> ap_rgetline() is no longer dealing with the final CRLF line correctly.
> Reverting that commit fixes things.
>
> Can anyone confirm that it's not just something on my end?
>
> --Jacob



-- 
Eric Covener
cove...@gmail.com

Re: Is 2.2.x broken?

2016-12-06 Thread William A Rowe Jr 
An associated patch was likely overlooked, please go ahead and revert.


On Dec 6, 2016 12:07 PM, "Jacob Champion"  wrote:

> I'm trying to test a backport to the 2.2.x tip, but it looks like r1758671
> might have busted our request parsing. Every test we have fails; I think
> ap_rgetline() is no longer dealing with the final CRLF line correctly.
> Reverting that commit fixes things.
>
> Can anyone confirm that it's not just something on my end?
>
> --Jacob
>

Is 2.2.x broken?

2016-12-06 Thread Jacob Champion 
I'm trying to test a backport to the 2.2.x tip, but it looks like 
r1758671 might have busted our request parsing. Every test we have 
fails; I think ap_rgetline() is no longer dealing with the final CRLF 
line correctly. Reverting that commit fixes things.


Can anyone confirm that it's not just something on my end?

--Jacob