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
