On Mon, Nov 14, 2016 at 8:07 PM, William A Rowe Jr <wr...@rowe-clan.net> wrote:
> On Mon, Nov 14, 2016 at 1:02 PM, <wr...@apache.org> wrote: > >> Author: wrowe >> Date: Mon Nov 14 19:02:29 2016 >> New Revision: 1769677 >> >> URL: http://svn.apache.org/viewvc?rev=1769677&view=rev >> Log: >> Promote one patch, propose one historically tangled patch >> >> + *) Propose default strict RFC7230 behavior, and HttpProtocolOptions >> directive >> + to relax or further constrain some behaviors. >> + trunk patches: too numerous to list, see >> + svn log --stop-on-copy http://svn.apache.org/repos/as >> f/httpd/httpd/branches/2.4.x-merge-http-strict/ >> + 2.4.x patch: see >> + svn diff -r1767912:HEAD http://svn.apache.org/repos/as >> f/httpd/httpd/branches/2.4.x-merge-http-strict/ >> + +1: wrowe >> > > Patch attached. There were some incremental comments to the merge branch > that may or may not still apply, please reiterate your point. > The patch suggested by rpluem has been integrated, a small broken diff to the httpd.h is corrected. So I'm attaching a refreshed net diff. > I'm aware of one possible failing of this backport which I would like to > fix on > 2.4 after the backport is in... compilers may complain of assignment of the > r->protocol to a const char*, we may need apr_pstrcpy's scattered about. > In practice I've found no error here, not even a warning in maintainer mode. > Thanks to everyone who has contributed to this work on trunk/ or simply > providing feedback on dev@. > > The earliest i will commit this backport is noon-ish Tuesday from Spain. > Looking forward to some reviewers of this net patch so we can move forward.
httpd-2.4-merge-http-strict.patch
Description: Binary data