bug-wget
[Top][All Lists]
Advanced

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

Re: [Bug-wget] [PATCH] certificate revocation lists (CRLs) #43501


From: Giuseppe Scrivano
Subject: Re: [Bug-wget] [PATCH] certificate revocation lists (CRLs) #43501
Date: Sat, 08 Nov 2014 13:00:13 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Tim Ruehsen <address@hidden> writes:

> On Friday 07 November 2014 09:26:58 Giuseppe Scrivano wrote:
>> Tim Ruehsen <address@hidden> writes:
>> > Here is a first patch (GnuTLS only) for review and comments (and playing
>> > around).
>> 
>> I think we should fail and avoid any connection instead of printing just
>> a warning as it seems from the code now.  Have you tested it with some
>> crl file?  Would be good to add some automatic tests for this new
>> feature.
>> 
>> > - Should we support complete directories ?
>> > - Should we allow more than one --crl-file option ?
>> 
>> We can add this later, but we need to ensure that wget fails now if more
>> --crl-file are passed so that the user knows it is not supported now.
>
> Amended patch.

thanks, the patch looks fine to me.

Regards,
Giuseppe



reply via email to

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