[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px
From: |
Tim Ruehsen |
Subject: |
Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px |
Date: |
Mon, 27 Oct 2014 11:51:24 +0100 |
User-agent: |
KMail/4.14.1 (Linux/3.16-2-amd64; KDE/4.14.1; x86_64; ; ) |
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 sending the request.
>
> The test was already using `sleep 1` to eliminate this race. I've bumped
> this up to 2 seconds. Now on my machine, the test no longer fails.
>
> If everyone is okay with this change, lets push it. If someone wants to
> change the number, please speak up.
Hi Darshit,
I made many test runs in the last time and never ever had
Test-proxied-https-auth.px fail here nor an a second machine (at home).
AFAIR, we did not identify a race condition (or let's say a design flaw). The
test suite starts the server and talks with it before starting Wget.
Please append your patch - I would like to further investigate what is really
going wrong (first of all I would like to reproduce the failure). I assume the
sleep just disguises a deeper problem that may pop up later in one or another
configuration.
Tim
signature.asc
Description: This is a digitally signed message part.
- [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px, Darshit Shah, 2014/10/27
- Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px,
Tim Ruehsen <=
- Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px, Tim Ruehsen, 2014/10/27
- Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px, Tim Ruehsen, 2014/10/27
- Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px, Tim Ruehsen, 2014/10/28
- Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px, Mike Frysinger, 2014/10/29
- Re: [Bug-wget] [Bug-Wget] Patch Test-proxied-https-auth.px, Tim Rühsen, 2014/10/29
- [Bug-wget] let's fix the openssl backend once and for all (was: Patch Test-proxied-https-auth.px), Giuseppe Scrivano, 2014/10/30
- Re: [Bug-wget] let's fix the openssl backend once and for all (was: Patch Test-proxied-https-auth.px), Darshit Shah, 2014/10/31
- Re: [Bug-wget] let's fix the openssl backend once and for all, Giuseppe Scrivano, 2014/10/31
- Re: [Bug-wget] let's fix the openssl backend once and for all, Tim Rühsen, 2014/10/31
- Re: [Bug-wget] let's fix the openssl backend once and for all (was: Patch Test-proxied-https-auth.px), Mike Frysinger, 2014/10/31