Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-06 Thread Yann Ylavic
On Fri, Jan 6, 2017 at 6:44 PM, Eric Covener wrote: > On Fri, Jan 6, 2017 at 12:06 PM, Jacob Champion wrote: >> >> IOW, the message/http payload body is allowed to be line-length limited, I >> assume because it's a message/* media type. But that doesn't

Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-06 Thread William A Rowe Jr
On Fri, Jan 6, 2017 at 11:44 AM, Eric Covener wrote: > On Fri, Jan 6, 2017 at 12:06 PM, Jacob Champion wrote: >>> Modified: >>> httpd/httpd/trunk/modules/http/http_filters.c >>> >>> Modified: httpd/httpd/trunk/modules/http/http_filters.c >>> URL: >>>

Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-06 Thread Eric Covener
On Fri, Jan 6, 2017 at 12:06 PM, Jacob Champion wrote: >> Modified: >> httpd/httpd/trunk/modules/http/http_filters.c >> >> Modified: httpd/httpd/trunk/modules/http/http_filters.c >> URL: >>

Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-06 Thread Jacob Champion
On 01/05/2017 04:31 AM, yla...@apache.org wrote: Author: ylavic Date: Thu Jan 5 12:31:48 2017 New Revision: 1777460 URL: http://svn.apache.org/viewvc?rev=1777460=rev Log: http: allow folding in check_headers(), still compliant with RFC 7230 (3.2.4). Modified:

Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-05 Thread William A Rowe Jr
On Thu, Jan 5, 2017 at 5:14 PM, Yann Ylavic wrote: > On Thu, Jan 5, 2017 at 11:49 PM, Yann Ylavic wrote: >> >> 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 >>

Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-05 Thread Yann Ylavic
On Thu, Jan 5, 2017 at 11:42 PM, Yann Ylavic wrote: > On Thu, Jan 5, 2017 at 11:08 PM, William A Rowe Jr > wrote: >> On Thu, Jan 5, 2017 at 4:05 PM, Eric Covener wrote: >>> Do we want this for the 2.2 release? >> >> I don't feel

Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-05 Thread Yann Ylavic
On Thu, Jan 5, 2017 at 11:08 PM, William A Rowe Jr wrote: > On Thu, Jan 5, 2017 at 4:05 PM, Eric Covener 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

Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-05 Thread William A Rowe Jr
On Thu, Jan 5, 2017 at 4:05 PM, Eric Covener 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.

Re: svn commit: r1777460 - /httpd/httpd/trunk/modules/http/http_filters.c

2017-01-05 Thread Eric Covener
Do we want this for the 2.2 release? On Thu, Jan 5, 2017 at 7:31 AM, wrote: > Author: ylavic > Date: Thu Jan 5 12:31:48 2017 > New Revision: 1777460 > > URL: http://svn.apache.org/viewvc?rev=1777460=rev > Log: > http: allow folding in check_headers(), still compliant with