Re: Fix error handling in be_tls_open_server() - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Fix error handling in be_tls_open_server()
Date
Msg-id ZQjxXuoKqt5ahkzv@paquier.xyz
Whole thread Raw
In response to Re: Fix error handling in be_tls_open_server()  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: Fix error handling in be_tls_open_server()
List pgsql-hackers
On Mon, Sep 18, 2023 at 02:35:28PM +0200, Daniel Gustafsson wrote:
> Certificates can be regenerated with the buildsystem, which ideally would apply
> to this cert as well, but if that's not feasible we can perhaps accept a static
> one with build information detailed in the README.  Having such a cert could
> for sure be interesting for testing.

WFM, but I'd prefer something that would be generated with the
makefile rules.  These are so handy when it comes to regenerate all
these certs..

> Awaiting resolution on this, I propose we go ahead with the original patch from
> this thread. Any objections to that?

I was wondering for a few seconds if you talked about the one posted
on [1], which would break the case where X509_NAME_get_text_by_NID()
fails if there's a valid bio, but you mean the one at the top of the
thread in [2], of course :)

One doubt that I have is if we shouldn't let X509_NAME_print_ex() be
as it is now, and not force a failure on the bio if this calls fails.

[1]: https://www.postgresql.org/message-id/E3921399-FAE7-4B1F-B1BF-B3357DDC9F19@yesql.se
[2]: https://www.postgresql.org/message-id/8db5374d-32e0-6abb-d402-40762511eff2@postgrespro.ru
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: 쿼리트릭스
Date:
Subject: Re: [ psql - review request ] review request for \d+ tablename, \d+ indexname indenting
Next
From: Michael Paquier
Date:
Subject: Re: XLog size reductions: smaller XLRec block header for PG17