RE: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-04 Thread Mladen Turk
Graham Leggett wrote: > > > >BalancerMember http://host1:port [balancer params] > >etc ... > > > > I just looked at the manual for - it defines config > that is common to a particular part of the URL space (it > seems to refer to the backend URL space). > Yes, I saw that too and l

RE: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-04 Thread Mladen Turk
I'm experiencing cca 2 hour mail delay on minotaur. Can you cc: to [EMAIL PROTECTED],hr jean-frederic clere wrote: > [mailto:[EMAIL PROTECTED] > Sent: Wednesday, August 04, 2004 3:20 PM > To: Tomcat Developers List > Subject: Re: cvs commit: jakarta-tomcat-connectors/ajp/pro

Re: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-04 Thread Graham Leggett
Mladen Turk wrote: IMO we could perhaps use exiting Proxy: BalancerMember ajp://host1:port [balancer params] etc... Or.. BalancerMember http://host1:port [balancer params] etc ... I just looked at the manual for - it defines config that is common to a particular part of the URL sp

Re: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-04 Thread jean-frederic clere
Mladen Turk wrote: [EMAIL PROTECTED] commited: new = apr_array_push(conf->aliases); -new->fake = f; -new->real = r; +new->fake = apr_pstrdup(cmd->pool, f); +new->real = apr_pstrdup(cmd->pool, r); There is no need to do that. The f is eventually set to cmd->p

RE: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-04 Thread Mladen Turk
[EMAIL PROTECTED] commited: > >new = apr_array_push(conf->aliases); > -new->fake = f; > -new->real = r; > +new->fake = apr_pstrdup(cmd->pool, f); > +new->real = apr_pstrdup(cmd->pool, r); > > There is no need to do that. The f is eventually set to cmd

RE: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-04 Thread Mladen Turk
jean-frederic clere wrote: > > Be carefull with our os: > +++ > #if (APR_MAJOR_VERSION < 1)^M > #undef apr_socket_create > #define apr_socket_create apr_socket_create_ex^M #endif^M > +++ > > It does not run on my OS(s): And I thought that I'm the 'style police' ;-). MT. smime.p7s Descript

Re: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-04 Thread jean-frederic clere
Hi, Be carefull with our os: +++ #if (APR_MAJOR_VERSION < 1)^M #undef apr_socket_create #define apr_socket_create apr_socket_create_ex^M #endif^M +++ It does not run on my OS(s): +++ [EMAIL PROTECTED]:~/apache20 > bin/apachectl start httpd: Syntax error on line 287 of /home/apache20/apache20/conf/h

RE: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-03 Thread Mladen Turk
Graham Leggett wrote: > > I have been mulling this over in my head for a bit. What do > you think of defining a group of load balancing servers that > serve requests, such as: > > >BalancerMember host1:port [balancer params] >BalancerMember host2:port [balancer params] > IMO we co

Re: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-03 Thread Graham Leggett
Mladen Turk wrote: Here is what I meant: ProxyPass /url balancer:number//[stickysessionname | none]/[nofailover] ProxyRemote balancer:number ://host-1(lbfactor) ProxyRemote balancer:number ://host-2(lbfactor) ... ProxyRemote balancer:number ://host-n(lbfactor) And (eventually) ... ProxyPassReverse

RE: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-03 Thread Mladen Turk
Graham Leggett wrote: > > [EMAIL PROTECTED] wrote: > > > Allow parsing lbfactor for ProxyPassReverse in the form > > ProxyPassReverse scheme://uri/path(lbfactor). > > The lbfactor is number 1...100 inside braces. > > Just a quick note - ProxyPassReverse is for dealing with the > conver

RE: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-03 Thread Mladen Turk
Graham Leggett wrote: > > Just a quick note - ProxyPassReverse is for dealing with the > conversion of Location headers from the backend to the frontend. > > A load balancing module should be using it's own directive > for this, not ProxyPassReverse. > Sure, but it's IMO the proper place fo

Re: cvs commit: jakarta-tomcat-connectors/ajp/proxy mod_proxy.c

2004-08-03 Thread Graham Leggett
[EMAIL PROTECTED] wrote: Allow parsing lbfactor for ProxyPassReverse in the form ProxyPassReverse scheme://uri/path(lbfactor). The lbfactor is number 1...100 inside braces. Just a quick note - ProxyPassReverse is for dealing with the conversion of Location headers from the backend to the fro