This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: setup.exe and cntlm proxy - setup gets stuck at mirrors.lst


for now I will try to convince cntlm people to support HTTP 1.0
cheers
pawel

On Sun, Aug 12, 2012 at 8:20 PM, Christopher Faylor wrote:
> On Sun, Aug 12, 2012 at 07:58:10PM +0200, Pawel Jasinski wrote:
>>On Sun, Aug 12, 2012 at 6:31 PM, Christopher Faylor  wrote:
>>> On Sun, Aug 12, 2012 at 03:48:16PM +0200, Pawel Jasinski wrote:
>>>>sorry for the cross post. I just realized setup.exe should be
>>>>discussed here and not in main cygwin mailing list
>>>>
>>>>I have created patch for setup.exe to make it work with cntlm proxy.
>>>>Upon investigation I have discovered that, setup.exe does not handle
>>>>well the situation when connection is not closed at the end of each
>>>>individual file (Keep-Alive) and gets stuck.
>>>>If the HTTP response from proxy contains Content-Length, patched
>>>>version uses provided length to detect end of file condition.
>>>>
>>>>What are the chances to include it in official setup.exe release?
>>>
>>> Nonexistent as it stands right now.
>>>
>>> Your submission lacks a ChangeLog and, more importantly, comments.  I
>>> have no idea why you're making the changes that you've made.  You're
>>> changing code that has worked for years so we really need some
>>> justification for why what you've done is necessary.
>>
>>never mind, somehow I assumed setup.exe is using HTTP 1.1. I am wrong,
>>setup is using HTTP 1.0
>>cntlm does not implement HTTP 1.0 correctly.
>>Given your comment, I assume you have no interest to switch to HTTP 1.1
>>
>>thanks for the prompt response
>
> You are somehow misreading my comment.  I didn't mention "HTTP anything"
> and I don't see any occurrence of 1.1 in the patch.
>
> If you have a patch which changes setup.exe to use HTTP 1.1 we could
> consider it but all of my previous comments still apply.
>
> cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]