Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-11-01 Thread Giuseppe Scrivano
Darshit Shah writes: > On 10/31, Giuseppe Scrivano wrote: >>Darshit Shah writes: >> >>> Ever since the Perl tests have been ported to the parallel test >>> harness, Test-proxied-https-auth.px has always failed for me. In a >>> prior thread, we'd identified the problem as a race between the HTTPS

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-31 Thread Darshit Shah
On 10/31, Giuseppe Scrivano wrote: Darshit Shah writes: Ever since the Perl tests have been ported to the parallel test harness, Test-proxied-https-auth.px has always failed for me. In a prior thread, we'd identified the problem as a race between the HTTPS server being set-up and Wget sending

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-31 Thread Giuseppe Scrivano
Tim Rühsen writes: > Did you get my email from today 09:43:19 ? > I fixed up the test (after reading how a *transparent* HTTPS proxy works). ops sorry, I've just seen it. > As Daniel mentioned, there of course these MITM HTTPS proxys. We need a test > for such a scenario as well. Maybe we shou

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-31 Thread Tim Rühsen
Am Freitag, 31. Oktober 2014, 16:32:28 schrieb Giuseppe Scrivano: > Darshit Shah writes: > > Ever since the Perl tests have been ported to the parallel test > > harness, Test-proxied-https-auth.px has always failed for me. In a > > prior thread, we'd identified the problem as a race between the HT

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-31 Thread Giuseppe Scrivano
Darshit Shah writes: > Ever since the Perl tests have been ported to the parallel test > harness, Test-proxied-https-auth.px has always failed for me. In a > prior thread, we'd identified the problem as a race between the HTTPS > server being set-up and Wget sending the request. > > The test was

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-31 Thread Tim Ruehsen
On Thursday 30 October 2014 14:43:58 Daniel Stenberg wrote: > On Thu, 30 Oct 2014, Tim Ruehsen wrote: > >> [*] = at least originally, until the MITM-ing proxies entered the scheme > >> and complicated matters, but I prefer to view that as messed up SSL and > >> not > >> "real" SSL =) > > > > Yes,

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-31 Thread Daniel Stenberg
On Thu, 30 Oct 2014, Tim Ruehsen wrote: [*] = at least originally, until the MITM-ing proxies entered the scheme and complicated matters, but I prefer to view that as messed up SSL and not "real" SSL =) Yes, however, Wget has to be able to work with these (if users request it). From how I u

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-30 Thread Tim Ruehsen
On Thursday 30 October 2014 10:55:49 Daniel Stenberg wrote: > On Thu, 30 Oct 2014, Tim Ruehsen wrote: > > How the test should work: > > - client open plain connection to proxy > > - client sends CONNECT request > > - server answers 200 OK > > - client/server change to SSL on the existing connection

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-30 Thread Daniel Stenberg
On Thu, 30 Oct 2014, Tim Ruehsen wrote: How the test should work: - client open plain connection to proxy - client sends CONNECT request - server answers 200 OK - client/server change to SSL on the existing connection (in the real world the proxy does this when it established the requested conne

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-30 Thread Tim Ruehsen
On Wednesday 29 October 2014 15:57:04 Mike Frysinger wrote: > On 28 Oct 2014 11:47, Tim Ruehsen wrote: > > I pushed it, since it fixes a bug anyways. > > it still seems to be racy for me -- passes only rarely. using wget 1.16 > with just the one fix 3eff3ad69a46364475e1f4abdf9412cfa87e3d6c. Meanw

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-29 Thread Tim Rühsen
Am Mittwoch, 29. Oktober 2014, 15:57:04 schrieb Mike Frysinger: > On 28 Oct 2014 11:47, Tim Ruehsen wrote: > > I pushed it, since it fixes a bug anyways. > > it still seems to be racy for me -- passes only rarely. using wget 1.16 > with just the one fix 3eff3ad69a46364475e1f4abdf9412cfa87e3d6c. >

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-29 Thread Mike Frysinger
On 28 Oct 2014 11:47, Tim Ruehsen wrote: > I pushed it, since it fixes a bug anyways. it still seems to be racy for me -- passes only rarely. using wget 1.16 with just the one fix 3eff3ad69a46364475e1f4abdf9412cfa87e3d6c. example failure log: $ cat Test-proxied-https-auth.log --2014-10-29 15:5

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-29 Thread Evangelos Foutras
On 29 October 2014 11:35, Tim Ruehsen wrote: > On Wednesday 29 October 2014 04:04:04 Evangelos Foutras wrote: >> The updated Test-proxied-https-auth.px fails very often for me. >> >> Log from the failing test is attached; using Perl 5.20.1 on Arch Linux. > > Ok, yes it is OpenSSL. > > I now can re

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-29 Thread Tim Ruehsen
On Wednesday 29 October 2014 04:04:04 Evangelos Foutras wrote: > The updated Test-proxied-https-auth.px fails very often for me. > > Log from the failing test is attached; using Perl 5.20.1 on Arch Linux. Ok, yes it is OpenSSL. I now can reproduce the problem - it happens randomly. I do not nee

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-29 Thread Tim Ruehsen
On Wednesday 29 October 2014 04:04:04 Evangelos Foutras wrote: > The updated Test-proxied-https-auth.px fails very often for me. > > Log from the failing test is attached; using Perl 5.20.1 on Arch Linux. Also, please add the output of src/wget --version I assume you are compiling/linkin

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-29 Thread Tim Ruehsen
On Wednesday 29 October 2014 04:04:04 Evangelos Foutras wrote: > The updated Test-proxied-https-auth.px fails very often for me. > > Log from the failing test is attached; using Perl 5.20.1 on Arch Linux. Hi Evangelos, On my Debian unstable machines (different hardware, same software) with perl

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-28 Thread Evangelos Foutras
The updated Test-proxied-https-auth.px fails very often for me. Log from the failing test is attached; using Perl 5.20.1 on Arch Linux. converted 'https://no.such.domain/needs-auth.txt' (ANSI_X3.4-1968) -> 'https://no.such.domain/needs-auth.txt' (UTF-8) --2014-10-29 02:21:13-- https://no.such.dom

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-28 Thread Tim Ruehsen
On Monday 27 October 2014 12:50:00 Tim Ruehsen wrote: > On Monday 27 October 2014 12:09:45 Tim Ruehsen wrote: > > On Monday 27 October 2014 11:51:24 Tim Ruehsen wrote: > > > On Sunday 26 October 2014 21:29:14 Darshit Shah wrote: > > > > Ever since the Perl tests have been ported to the parallel tes

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-27 Thread Tim Ruehsen
On Monday 27 October 2014 12:09:45 Tim Ruehsen wrote: > On Monday 27 October 2014 11:51:24 Tim Ruehsen wrote: > > On Sunday 26 October 2014 21:29:14 Darshit Shah wrote: > > > Ever since the Perl tests have been ported to the parallel test harness, > > > Test-proxied-https-auth.px has always failed

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-27 Thread Tim Ruehsen
On Monday 27 October 2014 11:51:24 Tim Ruehsen wrote: > On Sunday 26 October 2014 21:29:14 Darshit Shah wrote: > > Ever since the Perl tests have been ported to the parallel test harness, > > Test-proxied-https-auth.px has always failed for me. In a prior thread, > > we'd identified the problem as

Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-27 Thread Tim Ruehsen
On Sunday 26 October 2014 21:29:14 Darshit Shah wrote: > Ever since the Perl tests have been ported to the parallel test harness, > Test-proxied-https-auth.px has always failed for me. In a prior thread, we'd > identified the problem as a race between the HTTPS server being set-up and > Wget sendin

[Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px

2014-10-26 Thread Darshit Shah
Ever since the Perl tests have been ported to the parallel test harness, Test-proxied-https-auth.px has always failed for me. In a prior thread, we'd identified the problem as a race between the HTTPS server being set-up and Wget sending the request. The test was already using `sleep 1` to eli