Not to worry, we will appropriately change it. On Thu, 30 Apr 2020, Brian P Bockelman wrote: > @bbockelm commented on this pull request. > > > >> @@ -633,6 +635,19 @@ int XrdHttpProtocol::Process(XrdLink *lp) // We ignore the argument here > sbio = CreateBIO(Link); > BIO_set_nbio(sbio, 1); > ssl = SSL_new(sslctx); > + > + // On newer versions of OpenSSL, we use the periodically > + // updated store `verify_store` with the `x509_store_lock` > + // held; older OpenSSL versions are missing `SSL_set1_*_cert_store`, > + // so we simply rely on the global one in SSL_CTX. The latter > + // doesn't refresh the verify store, which causes issues with some > + // plugins. > +#if OPENSSL_VERSION_NUMBER >= 0x010100000L > > Argh - sorry, I haven't been able to sit down and get to this in the last two days (which have each had >6hrs of Zoom meetings...). You're right, this is too strict. > > -- > You are receiving this because you commented. > Reply to this email directly or view it on GitHub: > https://github.com/xrootd/xrootd/pull/1186#discussion_r418263100 -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/xrootd/xrootd/pull/1186#issuecomment-622118386 ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the XROOTD-DEV list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1