Re: Add test case for sslinfo

2019-08-16 Thread Peter Eisentraut
On 2019-07-08 10:18, Michael Paquier wrote: > On Mon, Jul 08, 2019 at 02:11:34PM +0800, Hao Wu wrote: >> Thank you for your quick response! I work on greenplum, and I didn't see >> this folder(src/test/ssl/ssl) before. >> I will add more certificates to test and resend again. > > Not having

Re: Add test case for sslinfo

2019-07-08 Thread Michael Paquier
On Mon, Jul 08, 2019 at 02:11:34PM +0800, Hao Wu wrote: > Thank you for your quick response! I work on greenplum, and I didn't see > this folder(src/test/ssl/ssl) before. > I will add more certificates to test and resend again. Not having duplicates would be nice. > Do you have any suggestion

Re: Add test case for sslinfo

2019-07-08 Thread Hao Wu
Hi Thomas, Thank you for your quick response! I work on greenplum, and I didn't see this folder(src/test/ssl/ssl) before. I will add more certificates to test and resend again. Do you have any suggestion about the missing PGDATA? Since the test needs to configure postgresql.conf, maybe there are

Re: Add test case for sslinfo

2019-07-07 Thread Thomas Munro
On Mon, Jul 8, 2019 at 2:59 PM Hao Wu wrote: > I see there is no test case for sslinfo. I have added a test case for it in > my project. Hi Hao Wu, Thanks! I see that you created a CF entry https://commitfest.postgresql.org/24/2203/. While I was scanning through the current CF looking for

Add test case for sslinfo

2019-07-07 Thread Hao Wu
Hi Hackers, I see there is no test case for sslinfo. I have added a test case for it in my project. Do you mind if I apply this test case to postgresql? Best regards, Hao Wu 0001-Add-certificates-keys-and-test-cases-for-contrib-ssl.patch Description: Binary data