[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] [PATCH] FTPS support

From: Ander Juaristi
Subject: Re: [Bug-wget] [PATCH] FTPS support
Date: Sat, 12 Sep 2015 01:58:57 +0200


I merged your first patch on top of mine and it all works well now. It
also passes all the tests. I also removed old code that appeared in my
previous patch (in connect.c) that should _not_ go to production.

You added the support for proxy for FTPS, which I ended up forgotting.
Well done!

Tell me if you still see something broken.

- AJ

On Fri, 2015-09-11 at 10:54 +0200, Tim Ruehsen wrote:
> Hi Ander,
> two things that I found.
> 1. [PATCH 0002] Some FTPS pieces seem to be missing.
> I went through the code by searching for SCHEME_FTP and "ftp" and added FTPS 
> stuff where it was missing. I did not think throughly about what I did - so 
> please just don't apply those changes blindly.
> With those changes, Wget tried to work recursively but hangs on the first 
> PASSIVE data transfer.
> 2. [PATCH 0003 ]'using_data_security' in getftp() is a local variable, reset 
> for each file, and only set if (prot != PROT_CLEAR). I turned the logic and 
> voila, recursion works. Again, think if this change might break something 
> else 
> that I didn't test.
> Regards, Tim

Attachment: 0001-Added-support-for-FTPS.patch
Description: Text Data

reply via email to

[Prev in Thread] Current Thread [Next in Thread]