Re: LB does not supports to rewrite location header with relative URLs ?

2014-06-18 Thread Imesh Gunaratne
Hi Melan, It's actually the Location header not the Host header. https://github.com/apache/stratos/blob/master/components/org.apache.stratos.load.balancer/src/main/java/org/apache/stratos/load/balancer/mediators/LocationReWriter.java Thanks On Wed, Jun 18, 2014 at 2:37 PM, Imesh Gunaratne wrot

Re: LB does not supports to rewrite location header with relative URLs ?

2014-06-18 Thread Imesh Gunaratne
Hi Melan, According to the current logic in LocationReWriter, even if the incoming Host header URL is relative the logic should not break. It will log the error and return without any problem. As a result the client will get the exact Host header returned by the member. However we could improve t

Re: LB does not supports to rewrite location header with relative URLs ?

2014-06-17 Thread Melan Nimesh
On Wed, Jun 18, 2014 at 8:16 AM, Isuru Haththotuwa wrote: > Hi Melan, > > A good find. Agreed that Location header should be properly supported. > Sorry if this is a dumb question, but could you please explain a bit more > what is the issue that occurs in this specific scenario you tested? > it'

Re: LB does not supports to rewrite location header with relative URLs ?

2014-06-17 Thread Isuru Haththotuwa
Hi Melan, A good find. Agreed that Location header should be properly supported. Sorry if this is a dumb question, but could you please explain a bit more what is the issue that occurs in this specific scenario you tested? On Wed, Jun 18, 2014 at 1:31 AM, Melan Nimesh wrote: > Hi Devs, > > I n

LB does not supports to rewrite location header with relative URLs ?

2014-06-17 Thread Melan Nimesh
Hi Devs, I noticed the $subject while subscribing to a PHP cartridge with a joomla repository. LB fails to rewrite location header with relative URLs with following error[1] and redirection not happens correctly. I think it's a valid requirement to support to location header with relative URLs, si