I'm -1 for interpretating invalid values. But +1 for alerting the admin of the invalid script/module/CGI. The new behavior was wrong, it should be set to now() for all invalid input IMHO
On Jul 21, 2016 5:20 PM, "Jacob Champion" <champio...@gmail.com> wrote: > On 07/03/2016 02:56 AM, Luca Toscano wrote: > >> Patch committed to trunk in http://svn.apache.org/r1751138 >> Updated backport proposal and trunk's CHANGES. >> > > Attached is an httpd-test patch with some regression cases for this > change. (I'm not sure what the review/commit policies are for that repo; > can anyone enlighten me?) > > Note that apr_date_parse_rfc() handles many more strange and semi-broken > cases than just RFC 822/1123. I am +1 for those but wanted to bring it up > to everyone's attention, just in case. > > --Jacob >