Re: [PATCH] Location header recovering to absoluteURI

I think this email doesn't arrive to the list. here we go again:

 I'm just visiting here since i've got an insanely amount of work, but
 i'v two comments.

 1) Eclipse?, really?

 2) I'm not pretty sure about "if (trans->port != 80)", it'll work fine
 in most cases but there is a border case when it'll not (Yeah, i know.
 I'm anal), the case is when you are going through ssl by port 80 (Yeah
 ok, shouldn't happen but you never know). I think the safest way is
 append port number always, it won't hurt anyway.

 M.


On Fri, Jan 23, 2009 at 6:10 PM, Martin Conte Mac Donell
<[email protected]> wrote:
> On Fri, Jan 23, 2009 at 3:15 PM, Stipe Tolj <[email protected]> wrote:
>> Hi list,
>>
>> we refuse to follow party defined absoluteURI Location header values. Which is
>> "correct" when being pedantic with the specs. But some mayor HTTP services, i.e.
>> Yahoo do send URI /xxx as Location header values. This patch ensures we are able
>> to recover from this circumstance, and continue the following.
>>
>> Please review, test and vote for committing to CVS HEAD.
>>
>> Stipe