Re: [Bug-wget] Test-504.py sometimes fails on slow machines

[Tim Ruehsen]

On Tuesday, February 14, 2017 4:20:26 PM CET Adam Sampson wrote:
> On Tue, Feb 14, 2017 at 04:03:13PM +0000, Adam Sampson wrote:
> > Your patch compiles, but doesn't fix the broken test. tcpdump traces
> > attached of the test succeeding and failing with the patch applied (it's
> > exactly the same behaviour as before).
>
> If it helps: here's a patch to the test server to add a delay after
> sending the headers -- this makes it always fail (on the second request)
> for me, even on a fast AMD64 machine.

Ok, refreshed my (poor) Python knowledge ;-)

Please try this patch, it works out for me even with your additional timing
(thanks for that !)

It fixes 504 handling in src/url.c and additionally adds Content-Length to
error responses that come with a body.

Regards, Tim

0001-Fix-504-status-handling.patch
Description: Text Data

signature.asc
Description: This is a digitally signed message part.