httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Plüm, Rüdiger, VF-Group <ruediger.pl...@vodafone.com>
Subject Re: apr_table_setn in mod_proxy_balancer.c
Date Mon, 09 Jul 2007 14:49:45 GMT


> -----Ursprüngliche Nachricht-----
> Von: jean-frederic clere 
> Gesendet: Montag, 9. Juli 2007 16:29
> An: dev@httpd.apache.org
> Betreff: Re: apr_table_setn in mod_proxy_balancer.c
> 
> 
> jean-frederic clere wrote:
> > Ruediger Pluem wrote:
> >>
> >> On 06/27/2007 05:09 PM, jean-frederic clere wrote:
> >>> Hi,
> >>>
> >>> In mod_proxy_balancer.c there is the following code:
> >>> +++
> >>>    /* Add the session route to request notes if present */
> >>>    if (route) {
> >>>        apr_table_setn(r->notes, "session-sticky", 
> (*balancer)->sticky);
> >>>        apr_table_setn(r->notes, "session-route", route);
> >>>
> >>>        /* Add session info to env. */
> >>>        apr_table_setn(r->subprocess_env,
> >>>                       "BALANCER_SESSION_STICKY", 
> (*balancer)->sticky);
> >>>        apr_table_setn(r->subprocess_env,
> >>>                       "BALANCER_SESSION_ROUTE", route);
> >>>    }
> >>> +++
> >>> "session-route" is SC_A_JVM_ROUTE in AJP but what are the 
> purpose of the
> >>> others?
> >>
> >> Does
> >>
> >> 
> http://httpd.apache.org/docs/2.2/en/mod/mod_proxy_balancer.htm
l#environment 
>>
>>
>> answer this question?
> 
> So I have to propose the attached patch to get 
> http://svn.apache.org/viewvc?view=rev&rev=551935 and 
> http://svn.apache.org/viewvc?view=rev&rev=550519 correct.,


Looks good to me.

Regards

Rüdiger


Mime
View raw message