On 1/4/18 15:41, Peter Eisentraut wrote: > On 12/28/17 02:19, Michael Paquier wrote: >> On Wed, Dec 27, 2017 at 09:27:40AM +0900, Michael Paquier wrote: >>> On Tue, Dec 26, 2017 at 03:28:09PM -0500, Peter Eisentraut wrote: >>>> On 12/22/17 03:10, Michael Paquier wrote: >>>>> Second thoughts on 0002 as there is actually no need to move around >>>>> errorMessage if the PGconn* pointer is saved in the SCRAM status data >>>>> as both are linked. The attached simplifies the logic even more. >>>>> >>>> >>>> That all looks pretty reasonable. >>> >>> Thanks for the review. Don't you think that the the refactoring >>> simplifications should be done first though? This would result in >>> producing the patch set in reverse order. I'll be fine to produce them >>> if need be. >> >> Well, here is a patch set doing the reverse operation: refactoring does >> first in 0001 and support for tls-server-end-point is in 0002. Hope this >> helps. > > committed
Some hosts don't seem to have X509_get_signature_nid(). Looking into that ... -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
