Re: svn commit: r1836381 - in /httpd/httpd/trunk: ./ include/ modules/proxy/ modules/proxy/balancers/

2018-08-22 Thread William A Rowe Jr
On Fri, Jul 20, 2018 at 2:27 PM,  wrote:

> Author: rpluem
> Date: Fri Jul 20 19:27:31 2018
> New Revision: 1836381
>
> URL: http://svn.apache.org/viewvc?rev=1836381&view=rev
> Log:
> * mod_proxy: Remove load order and link dependency between mod_lbmethod_*
>   modules and mod_proxy by providing mod_proxy's
> ap_proxy_balancer_get_best_worker
>   as an optional function.
>

Thanks RĂ¼diger!

Not sure if I missed this, but did you want to advance this regression
fix on the 2.4.x branch? That seems to be one showstopper to getting
us to a maintenance/stabilizing release.


Re: svn commit: r1836381 - in /httpd/httpd/trunk: ./ include/ modules/proxy/ modules/proxy/balancers/

2018-07-20 Thread William A Rowe Jr
On Fri, Jul 20, 2018, 15:22 Ruediger Pluem  wrote:

>
> BTW: We have the same load order issue issue with the following proxy
> modules:
>
> mod_proxy_connect
> mod_proxy_ftp
> mod_proxy_http
> mod_proxy_fcgi
> mod_proxy_scgi
> mod_proxy_fdpass
> mod_proxy_ajp
> mod_proxy_balancer
> mod_proxy_express
>

Correct. However this isn't a regression, and mod_proxy is an apparent
prerequisite to any of these, which further and thankfully sorts first. The
lbmethod providers differed in all three respects.

It might be interesting to solve in a future enhancement release, but the
number of dependencies likely makes this unrealistic.


Re: svn commit: r1836381 - in /httpd/httpd/trunk: ./ include/ modules/proxy/ modules/proxy/balancers/

2018-07-20 Thread Ruediger Pluem



On 07/20/2018 09:27 PM, [email protected] wrote:
> Author: rpluem
> Date: Fri Jul 20 19:27:31 2018
> New Revision: 1836381
> 
> URL: http://svn.apache.org/viewvc?rev=1836381&view=rev
> Log:
> * mod_proxy: Remove load order and link dependency between mod_lbmethod_*
>   modules and mod_proxy by providing mod_proxy's 
> ap_proxy_balancer_get_best_worker
>   as an optional function.
> 
> PR: 62557
> 

BTW: We have the same load order issue issue with the following proxy modules:

mod_proxy_connect
mod_proxy_ftp
mod_proxy_http
mod_proxy_fcgi
mod_proxy_scgi
mod_proxy_fdpass
mod_proxy_ajp
mod_proxy_balancer
mod_proxy_express

Regards

RĂ¼diger