Re: [PATCH 2/3] http: add support for disabling SSL revocation checks in cURL
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Oct 16, 2018, 12:21 UTC
- Message-ID
- <nycvar.QRO.7.76.6.1810161421340.4546@tvgsbejvaqbjf.bet>
- In-Reply-To
- <CAPig+cQFb3S0Lm+huUZDN4aw9rWwinh0iZp12ss1zVKpJ=2MdA@mail.gmail.com>
Hi Eric,
On Mon, 15 Oct 2018, Eric Sunshine wrote:
Show 21 quoted lines
> On Mon, Oct 15, 2018 at 6:14 AM Brendan Forster via GitGitGadget
> <gitgitgadget@gmail.com> wrote:
> > This config value is only used if http.sslBackend is set to "schannel",
> > which forces cURL to use the Windows Certificate Store when validating
> > server certificates associated with a remote server.
> >
> > This is only supported in cURL 7.44 or later.
> > [...]
> > Signed-off-by: Brendan Forster <github@brendanforster.com>
> > ---
> > diff --git a/http.c b/http.c
> > @@ -811,6 +818,16 @@ static CURL *get_curl_handle(void)
> > + if (http_ssl_backend && !strcmp("schannel", http_ssl_backend) &&
> > + !http_schannel_check_revoke) {
> > +#if LIBCURL_VERSION_NUM >= 0x072c00
> > + curl_easy_setopt(result, CURLOPT_SSL_OPTIONS, CURLSSLOPT_NO_REVOKE);
> > +#else
> > + warning("CURLSSLOPT_NO_REVOKE not applied to curl SSL options because\n"
> > + "your curl version is too old (>= 7.44.0)");
>
> This message is confusing. If your curl is too old, shouldn't the ">=" be a "<"?Absolutely correct. Will fix, Dscho
> > > +#endif > > + } >