[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-wget] wget - cntlm incompatibility (wget 1.13 onwards)
From: |
Schroeteler, Thorsten |
Subject: |
Re: [Bug-wget] wget - cntlm incompatibility (wget 1.13 onwards) |
Date: |
Mon, 16 Jun 2014 13:11:57 +0200 |
Hello Giuseppe,
Yes it works like you suggest:
request_set_header (req, "Connection", "Keep-Alive", rel_none);
request_set_header (req, "Proxy-Connection", "Keep-Alive", rel_none);
The important thing seems that "Connection" is not closed.
Cheers,
Thorsten
-----Original Message-----
From: Giuseppe Scrivano [mailto:address@hidden
Sent: Monday, June 16, 2014 12:26 PM
To: Darshit Shah
Cc: Schroeteler, Thorsten; bug-wget
Subject: Re: [Bug-wget] wget - cntlm incompatibility (wget 1.13 onwards)
Darshit Shah <address@hidden> writes:
>> In wget-1.13 if I replace the two request_set_header calls in http.c
>> (line 1683,1684) with one
>> for "Connection" "Keep-Alive" then the download is successful:
>>
>> if (inhibit_keep_alive)
>> request_set_header (req, "Connection", "Close", rel_none);
>> else
>> {
>> if (proxy == NULL)
>> request_set_header (req, "Connection", "Keep-Alive",
rel_none);
>> else
>> {
>> //request_set_header (req, "Connection", "Close",
rel_none);
>> //request_set_header (req, "Proxy-Connection",
"Keep-Alive",
>> rel_none);
>> request_set_header (req, "Connection", "Keep-Alive",
>> rel_none);
>> }
>> }
>>
>>
>> I don't know whether this is rather a wget or an cntlm issue. I will
>> post this to the cntlm mailing list as well.
>> Any help is appreciated (it would be nice if we don't need to patch
wget
>> in future ...).
>>
>
> Hi Thorsten,
>
> Sorry for the delay in responding and thanks for reporting this to us.
> This does indeed look like a bug to me. When inhibit_keep_alive is not
> set, Wget should indeed be sending a Connection: Keep-Alive header and
> not close the connection. Unless there is some specification about
> working with proxies that I forget.
>
> I think this is a bug and needs to be fixed with the above provided
> solution. Does anyone here object?
>From the ChangeLog file, it seems that Proxy-Connection was introduced
to fix a problem with some HTTP/1.0 proxies.
Thorsten, does it work if we leave both headers set like the following?
request_set_header (req, "Connection", "Keep-Alive", rel_none);
request_set_header (req, "Proxy-Connection", "Keep-Alive", rel_none);
Regards,
Giuseppe