On 12/26/17 15:53, Peter Eisentraut wrote: > This patch looks reasonable to me. I have also seen occasional requests > for this in the field. > > If someone could test this on Windows, I think we could move ahead with it.
A small point on the test changes. You change the test under "diagnostic message", but I'm not sure why. Do the changes invalidate the existing test? We should probably also add another "note" call to introduce the LDAPS tests section. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services