It looks like the whole retry was removed :(
> On Feb 10, 2016, at 2:21 PM, Ruediger Pluem <[email protected]> wrote:
>
>
>
> On 02/10/2016 12:38 AM, [email protected] wrote:
>> Author: ylavic
>> Date: Tue Feb 9 23:38:59 2016
>> New Revision: 1729507
>>
>> URL: http://svn.apache.org/viewvc?rev=1729507&view=rev
>> Log:
>> mod_proxy: axe negative "ping" parameter setting and handling.
>> This used to check for the backend connection readability only (instead of
>> the full ping/100-continue round-trip), but the case is already handled by
>> ap_proxy_connect_backend() which is always called.
>>
>> Modified:
>> httpd/httpd/trunk/modules/http2/mod_proxy_http2.c
>> httpd/httpd/trunk/modules/proxy/mod_proxy.c
>> httpd/httpd/trunk/modules/proxy/mod_proxy.h
>> httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c
>> httpd/httpd/trunk/modules/proxy/mod_proxy_http.c
>> httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c
>>
>
>> Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c
>> URL:
>> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c?rev=1729507&r1=1729506&r2=1729507&view=diff
>> ==============================================================================
>> --- httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c (original)
>> +++ httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c Tue Feb 9 23:38:59
>> 2016
>
>> @@ -463,15 +462,13 @@ static int proxy_wstunnel_handler(reques
>> backend->is_ssl = is_ssl;
>> backend->close = 0;
>>
>> - retry = 0;
>> - while (retry < 2) {
>> + do { /* while (0): break out */
>> char *locurl = url;
>> /* Step One: Determine Who To Connect To */
>> status = ap_proxy_determine_connection(p, r, conf, worker, backend,
>> uri, &locurl, proxyname,
>> proxyport,
>> server_portstr,
>> sizeof(server_portstr));
>> -
>> if (status != OK)
>> break;
>>
>> @@ -495,8 +492,7 @@ static int proxy_wstunnel_handler(reques
>> /* Step Three: Process the Request */
>> status = proxy_wstunnel_request(p, r, backend, worker, conf, uri,
>> locurl,
>> server_portstr, scheme);
>> - break;
>> - }
>> + } while (0);
>
> Do you keep this loop just to keep the formating?
>
> Regards
>
> Rüdiger
>
