On 06/28/2016 04:26 PM, Ruediger Pluem wrote: > > > On 06/28/2016 01:19 PM, [email protected] wrote: >> Author: ylavic >> Date: Tue Jun 28 11:19:36 2016 >> New Revision: 1750474 >> >> URL: http://svn.apache.org/viewvc?rev=1750474&view=rev >> Log: >> mod_proxy: follow up to r1750392. >> Avoid double checking the connection in ap_proxy_connect_backend() when >> ap_proxy_check_backend() says it is up and good to go. >> >> This can be done by moving the PROXY_WORKER_IS_USABLE() check in >> ap_proxy_check_backend(), since it is called by ap_proxy_connect_backend(), >> and not calling the latter if the former succeeded (for the modules using >> it). > > IMHO this is a bad idea, because ap_proxy_connect_backend does more then just > checking the socket, even if it is connected. > > See lines 2915 till 2955.
Typed faster than reading. I see this is already addressed. Regards Rüdiger
