Upon inspection it looks good... doing some further testing
but so far, +1

Thx!
> On Dec 12, 2016, at 11:18 AM, Yann Ylavic <ylavic....@gmail.com> wrote:
> 
> On Mon, Dec 12, 2016 at 3:32 PM, Eric Covener <cove...@gmail.com> wrote:
>> On Mon, Dec 12, 2016 at 6:25 AM, Yann Ylavic <ylavic....@gmail.com> wrote:
>>> Tested with "Header set 'X-Bad' '<something with CR>'" with no redirect 
>>> loop.
>> 
>> w/ 'Header always set...' this still repeats
> 
> Right, with r1773812 it looks good.
> We'd only recurse if not in internal redirect already, otherwise fall
> through with:
>    HTTP/1.1 500 Internal Server Error
>    Date: Mon, 12 Dec 2016 16:11:59 GMT
>    Server: Apache/2.5.0-dev (Unix) OpenSSL/1.0.2g
>    Content-Length: 0
>    Connection: close
> 
> WDYT?

Reply via email to