RE: server/util_expr_(parse|scan) generated sources files

2014-12-10 Thread Plüm , Rüdiger , Vodafone Group
+1 Regards Rüdiger -Original Message- From: Yann Ylavic [mailto:ylavic@gmail.com] Sent: Dienstag, 9. Dezember 2014 22:28 To: httpd Subject: Re: server/util_expr_(parse|scan) generated sources files On Tue, Dec 9, 2014 at 10:01 PM, Gregg Smith g...@gknw.net wrote: Maybe we

[PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Jan Kaluža
Hi, I've found out that following configuration does not work as expected: Proxy balancer://a ... /Proxy VirtualHost *:80 ProxyPass / balancer://a stickysession=JSESSIONID|jsessionid /VirtualHost In this case, two proxy_balancers are created. The first one in Proxy section in the main

Re: [PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Yann Ylavic
Hi, didn't look at the patch yet, but the workaround for this is usually to use ProxySet in the Proxy block. I agree that it would be nice to have these parameters merged, though. Regards, Yann. On Wed, Dec 10, 2014 at 1:25 PM, Jan Kaluža jkal...@redhat.com wrote: Hi, I've found out that

RE: [PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Plüm , Rüdiger , Vodafone Group
But this way we lose the base ones that are not touched in the virtual host and e.g. are only used by rewriterules. So we should transfer the base ones to the merged array in any case and update them where needed. Isn't the config merge on a critical path with every request? So double for loops

Re: [PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Yann Ylavic
On Wed, Dec 10, 2014 at 1:49 PM, Plüm, Rüdiger, Vodafone Group ruediger.pl...@vodafone.com wrote: Isn't the config merge on a critical path with every request? I don't think so, my understanding is that the *server* config merge is before post_config. Regards, Yann.

Re: [PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Yann Ylavic
On Wed, Dec 10, 2014 at 1:52 PM, Yann Ylavic ylavic@gmail.com wrote: didn't look at the patch yet, but the workaround for this is usually to use ProxySet in the Proxy block. This is not (at all) equivalent to a merge, please ignore this... Regards, Yann.

Re: [PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Jan Kaluža
On 12/10/2014 01:49 PM, Plüm, Rüdiger, Vodafone Group wrote: But this way we lose the base ones that are not touched in the virtual host and e.g. are only used by rewriterules. So we should transfer the base ones to the merged array in any case and update them where needed. Hm, you are

Re: [PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Yann Ylavic
On Wed, Dec 10, 2014 at 2:21 PM, Jan Kaluža jkal...@redhat.com wrote: On 12/10/2014 01:49 PM, Plüm, Rüdiger, Vodafone Group wrote: Isn't the config merge on a critical path with every request? So double for loops always worry me a little bit from performance point of view. I think that

Re: [PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Jan Kaluža
On 12/10/2014 02:50 PM, Yann Ylavic wrote: On Wed, Dec 10, 2014 at 2:21 PM, Jan Kaluža jkal...@redhat.com wrote: On 12/10/2014 01:49 PM, Plüm, Rüdiger, Vodafone Group wrote: Isn't the config merge on a critical path with every request? So double for loops always worry me a little bit from

Re: [PATCH] Balancers, VirtualHost and ProxyPass

2014-12-10 Thread Ruediger Pluem
On 12/10/2014 02:21 PM, Jan Kaluža wrote: On 12/10/2014 01:49 PM, Plüm, Rüdiger, Vodafone Group wrote: But this way we lose the base ones that are not touched in the virtual host and e.g. are only used by rewriterules. So we should transfer the base ones to the merged array in any case and