Re: svn commit: r1884505 - in /httpd/httpd/trunk: changes-entries/pr64339.txt modules/filters/mod_xml2enc.c

2020-12-16 Thread Yann Ylavic
On Wed, Dec 16, 2020 at 6:36 PM Yann Ylavic wrote: > > On Wed, Dec 16, 2020 at 5:23 PM wrote: > > > > -/* only act if starts-with "text/" or contains "xml" */ > > -if (strncmp(ctype, "text/", 5) && !strstr(ctype, "xml")) { > > +/* only act if starts-with "text/" or contains "+xml"

Re: svn commit: r1884505 - in /httpd/httpd/trunk: changes-entries/pr64339.txt modules/filters/mod_xml2enc.c

2020-12-16 Thread Yann Ylavic
On Wed, Dec 16, 2020 at 5:23 PM wrote: > > -/* only act if starts-with "text/" or contains "xml" */ > -if (strncmp(ctype, "text/", 5) && !strstr(ctype, "xml")) { > +/* only act if starts-with "text/" or contains "+xml" */ > +if (strncmp(ctype, "text/", 5) && !strstr(ctype,

Re: svn commit: r1884505 - in /httpd/httpd/trunk: changes-entries/pr64339.txt modules/filters/mod_xml2enc.c

2020-12-16 Thread Nick Kew
> On 16 Dec 2020, at 17:47, Yann Ylavic wrote: > > On Wed, Dec 16, 2020 at 6:36 PM Yann Ylavic wrote: >> >> On Wed, Dec 16, 2020 at 5:23 PM wrote: >>> >>> -/* only act if starts-with "text/" or contains "xml" */ >>> -if (strncmp(ctype, "text/", 5) && !strstr(ctype, "xml")) { >>>

Re: svn commit: r1884280 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_ftp.c modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2020-12-16 Thread Roy T. Fielding
> On Dec 16, 2020, at 2:52 AM, Graham Leggett wrote: > > On 12 Dec 2020, at 01:59, Roy T. Fielding > wrote: > >> That is too many questions. The purpose of the cache requirement is so that >> the cache >> does not deliver a non-validated entry after receiving a

Re: svn commit: r1884456 - /httpd/httpd/trunk/CHANGES

2020-12-16 Thread Yann Ylavic
On Tue, Dec 15, 2020 at 9:36 PM Ruediger Pluem wrote: > > On 12/15/20 4:38 PM, Yann Ylavic wrote: > > > > When should we do that? I suppose anytime on trunk and when merging on > > 2.4.x (or at worst when releasing)? > Correct. Any time on trunk or if you don't want to have it on the trunk >

Re: svn commit: r1884280 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_ftp.c modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2020-12-16 Thread Graham Leggett
On 12 Dec 2020, at 01:59, Roy T. Fielding wrote: > That is too many questions. The purpose of the cache requirement is so that > the cache > does not deliver a non-validated entry after receiving a failed validation. > It doesn't really > matter what code is returned so long as it is 5xx, so