Re: HTTP_MISDIRECTED_REQUEST

2015-08-28 Thread Stefan Eissing
Am 28.08.2015 um 03:37 schrieb Roy T. Fielding field...@gbiv.com: +if (r-connection-keepalives 0) { +return HTTP_MISDIRECTED_REQUEST; +} return HTTP_BAD_REQUEST; } } IIRC, it is applicable to

Re: HTTP_MISDIRECTED_REQUEST

2015-08-28 Thread Stefan Eissing
Am 28.08.2015 um 10:32 schrieb Ruediger Pluem rpl...@apache.org: On 08/28/2015 09:32 AM, Stefan Eissing wrote: Am 28.08.2015 um 03:37 schrieb Roy T. Fielding field...@gbiv.com: +if (r-connection-keepalives 0) { +return HTTP_MISDIRECTED_REQUEST; +

Re: HTTP_MISDIRECTED_REQUEST

2015-08-28 Thread Ruediger Pluem
On 08/28/2015 09:32 AM, Stefan Eissing wrote: Am 28.08.2015 um 03:37 schrieb Roy T. Fielding field...@gbiv.com: +if (r-connection-keepalives 0) { +return HTTP_MISDIRECTED_REQUEST; +} return HTTP_BAD_REQUEST;

Re: regression in r1698239

2015-08-28 Thread Eric Covener
On Fri, Aug 28, 2015 at 8:54 AM, Stefan Eissing stefan.eiss...@greenbytes.de wrote: Change 1698239 brings t/apache/pr17629.t into an endless loop now. I built on r1698322 in trunk (OSX + Ubuntu). I revert the change now. A certain covener might want to look into this. ;-) Cheers, Thanks,

Re: HTTP_MISDIRECTED_REQUEST

2015-08-28 Thread Eric Covener
On Fri, Aug 28, 2015 at 9:26 AM, Stefan Eissing stefan.eiss...@greenbytes.de wrote: If this works, one could think about introducing some kind of equivalence number to speed up the checking, since in certain HTTP/2 setups there might be a good percentage of requests requiting this

Re: regression in r1698239

2015-08-28 Thread Eric Covener
Thanks -- back in 1698334 with loops that actually move. On Fri, Aug 28, 2015 at 8:55 AM, Eric Covener cove...@gmail.com wrote: On Fri, Aug 28, 2015 at 8:54 AM, Stefan Eissing stefan.eiss...@greenbytes.de wrote: Change 1698239 brings t/apache/pr17629.t into an endless loop now. I built on

Re: HTTP_MISDIRECTED_REQUEST

2015-08-28 Thread Stefan Eissing
As long as we are waiting for the *real* experts to chime in, I took a first stab refactoring the code a tiny bit and adding an additional check against the selected server_rec. If the r-hostname does not match the SNI servername, the r-hostname is used to check if it matches the

regression in r1698239

2015-08-28 Thread Stefan Eissing
Change 1698239 brings t/apache/pr17629.t into an endless loop now. I built on r1698322 in trunk (OSX + Ubuntu). I revert the change now. A certain covener might want to look into this. ;-) Cheers, Stefan green/bytes GmbH Hafenweg 16, 48155 Münster, Germany Phone: +49 251 2807760.

Re: regression in r1698239

2015-08-28 Thread Stefan Eissing
Works fine here. Thanks! Am 28.08.2015 um 15:14 schrieb Eric Covener cove...@gmail.com: Thanks -- back in 1698334 with loops that actually move. On Fri, Aug 28, 2015 at 8:55 AM, Eric Covener cove...@gmail.com wrote: On Fri, Aug 28, 2015 at 8:54 AM, Stefan Eissing

Re: svn commit: r1610674 - in /httpd/httpd/trunk: include/ap_mmn.h include/httpd.h modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c server/util.c

2015-08-28 Thread Christophe JAILLET
Hi, spotted while looking at https://raw.githubusercontent.com/icing/mod_h2/master/sandbox/httpd/patches/core-protocols.patch which include it. CJ Le 15/07/2014 14:27, jor...@apache.org a écrit : Author: jorton Date: Tue Jul 15 12:27:00 2014 New Revision: 1610674 URL:

Re: [RFC] Enable OCSP Stapling by default in httpd trunk

2015-08-28 Thread Jeff Trawick
On Mon, Jul 13, 2015 at 3:08 AM, Ruediger Pluem rpl...@apache.org wrote: On 07/11/2015 08:55 PM, William A Rowe Jr wrote: If you are suggesting we shouldn't change the compiled-in default, I can agree, POLS (Principal Of Least Surprise). If you are suggesting the default config