On Thu, Jan 5, 2017 at 11:42 PM, Yann Ylavic <ylavic....@gmail.com> wrote: > On Thu, Jan 5, 2017 at 11:08 PM, William A Rowe Jr <wr...@rowe-clan.net> > wrote: >> On Thu, Jan 5, 2017 at 4:05 PM, Eric Covener <cove...@gmail.com> wrote: >>> Do we want this for the 2.2 release? >> >> I don't feel strongly about this. >> >> It is such an unusual edge case (I believe Yann pointed out it was a custom >> module he was working around) that it should rarely be seen in the wild. >> >> I'd be fine if we want to accelerate this into 2.2.32, or get 2.2.32 >> out and then >> backport to have the patch available for the very few who are impacted. > > Agreed, I wouldn't push it, and can live with a my own patching. > Possibly another module/cgi uses that CRLF to LWS trick though, but > not that I'm aware of so..
But if any of you fears a possible regression for older 2.2.x apps (I see now that Eric included a test, I personnaly tested it this afternoon with a custom integration suite too), I'm happy to propose and vote it.
