Re: svn commit: r1832609 - in /httpd/httpd/branches/2.4.x: ./ docs/manual/howto/ docs/manual/mod/ modules/proxy/ modules/proxy/balancers/

2018-07-20 Thread William A Rowe Jr
Looks like a good direction. From PR62557, the observed modules to be adjusted, to consume the new opt fn are; --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -423,6 +424,9 @@ SET(mod_http2_extra_sources modules/http2/h2_task.cmodules/http2/h2_util.c modules/http2/h2_workers.c )

Re: svn commit: r1832609 - in /httpd/httpd/branches/2.4.x: ./ docs/manual/howto/ docs/manual/mod/ modules/proxy/ modules/proxy/balancers/

2018-07-20 Thread Yann Ylavic
On Fri, Jul 20, 2018 at 2:56 PM, Ruediger Pluem wrote: > > So maybe some checking is due there as well if the conditional fetching > should be removed? +1 :)

Re: svn commit: r1832609 - in /httpd/httpd/branches/2.4.x: ./ docs/manual/howto/ docs/manual/mod/ modules/proxy/ modules/proxy/balancers/

2018-07-20 Thread Ruediger Pluem
On 07/20/2018 02:45 PM, Yann Ylavic wrote: > On Fri, Jul 20, 2018 at 12:13 PM, Ruediger Pluem wrote: >> >> Something like the attached? The mod_lbmethod_byrequests.c part needs to be >> done for lb modules though. > > +/* post_config hook: */ > +static int

Re: svn commit: r1832609 - in /httpd/httpd/branches/2.4.x: ./ docs/manual/howto/ docs/manual/mod/ modules/proxy/ modules/proxy/balancers/

2018-07-20 Thread Yann Ylavic
On Fri, Jul 20, 2018 at 12:13 PM, Ruediger Pluem wrote: > > Something like the attached? The mod_lbmethod_byrequests.c part needs to be > done for lb modules though. +/* post_config hook: */ +static int lbmethod_byrequests_post_config(apr_pool_t *pconf, apr_pool_t *plog, +apr_pool_t

Re: svn commit: r1832609 - in /httpd/httpd/branches/2.4.x: ./ docs/manual/howto/ docs/manual/mod/ modules/proxy/ modules/proxy/balancers/

2018-07-20 Thread Ruediger Pluem
On 07/19/2018 08:24 PM, William A Rowe Jr wrote: > On Thu, May 31, 2018 at 8:24 AM, mailto:j...@apache.org>> > wrote: > > Author: jim > Date: Thu May 31 13:24:04 2018 > New Revision: 1832609 > > URL: http://svn.apache.org/viewvc?rev=1832609=rev >