On Wed, Jan 11, 2023 at 6:27 PM Jacob Champion <jchamp...@timescale.com> wrote:
> On Wed, Jan 11, 2023 at 6:37 AM Jelte Fennema <postg...@jeltef.nl> wrote: > > > > LGTM. As far as I can tell this is ready for a committer. > > Thanks for the reviews! > Sorry to jump in (very) late in this game. So first, I like this general approach :) It feels icky to have to add configure tests just to make a test work. But I guess there isn't really a way around that if we want to test the full thing. However, shouldn't we be using X509_get_default_cert_file_env() to get the name of the env? Granted it's unlikely to be anything else, but if it's an API you're supposed to use. (In an ideal world that function would not return anything in LibreSSL but I think it does include something, and then just ignores it?) -- Magnus Hagander Me: https://www.hagander.net/ <http://www.hagander.net/> Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>
