httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jean-frederic clere <jfcl...@gmail.com>
Subject Re: svn commit: r550519 - in /httpd/httpd/trunk: docs/manual/mod/mod_proxy.html.en docs/manual/mod/mod_proxy.xml modules/proxy/mod_proxy.c modules/proxy/mod_proxy.h modules/proxy/mod_proxy_balancer.c
Date Wed, 27 Jun 2007 07:05:20 GMT
Ruediger Pluem wrote:
> On 06/26/2007 06:46 PM, jean-frederic clere wrote:
>   
>> Ruediger Pluem wrote:
>>     
>
>   
>>> Ok, furthermore I think we need to adjust the proxy_status_hook to
>>> actually display the string the user configured and not only the path
>>> for the cookie. The same is true for the balancer manager (display wise).
>>> Missing to split the string at the | in the balancer manager when you
>>> enter new data for sticky does not really worry me because setting
>>> a new sticky for the balancer (like all other balancer parameters)
>>> does not work anyway. We should really disable this.
>>>   
>>>       
>> How? Keep the balancer-manager page as it is but "disable" that part.
>> (form with fields read only and submit disabled).
>>     
>
> I am thinking more about removing it completely like in the attached patch, because
>
> 1. All information is also displayed without the form.
> 2. If it turns out that we need the form for the balancer again, because we adjusted
>    the balancer data structures in a way that changes via the balancer-manager work,
>    we can fetch it back from svn. IMHO no need to keep "dead" code.
>   
Ok I will commit your patch together with what was missing in r550519.

Cheers
Jean-Frederic
>
> Regards
>
> RĂ¼diger
>
>
>
>   


Mime
View raw message