<div dir="ltr">Hi Tatsuo,<div>Please find the attached updated patch with following changes:</div><div>1. Updated the description of '<i>ssl_crl_file'</i> configuration variable.</div><div>2. Updated test case '024.cert_auth' which verify valid CRL and invalid CRL ( CRL with revocation entry ) </div><div><br></div><div>Regards,</div><div>Umar Hayat</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 13, 2020 at 3:43 AM Tatsuo Ishii <<a href="mailto:ishii@sraoss.co.jp">ishii@sraoss.co.jp</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> I just followed the description pattern used for other ssl variables. We<br>
> can use PostgreSQL doc if we remove following two line from that:<br>
> "Relative paths are relative to the data<br>
> directory. This parameter can only be set in the postgresql.conf file<br>
> or on the server command line.<br>
> "<br>
<br>
Sounds good to me.<br>
<br>
> - It would be nice to include regression test patch. See<br>
>> src/test/023.ssl_connection for an example.<br>
>><br>
> <br>
> Sure, I will create and send test patch in src/test/023.ssl_connection.<br>
> I will try to generate CRL file for existing certification file in this<br>
> this test. If thats not possible, then I have to generate new certification<br>
> and CRL file.<br>
<br>
Thank you. Looking forward to the new patch.<br>
<br>
Best regards,<br>
--<br>
Tatsuo Ishii<br>
SRA OSS, Inc. Japan<br>
English: <a href="http://www.sraoss.co.jp/index_en.php" rel="noreferrer" target="_blank">http://www.sraoss.co.jp/index_en.php</a><br>
Japanese:<a href="http://www.sraoss.co.jp" rel="noreferrer" target="_blank">http://www.sraoss.co.jp</a><br>
</blockquote></div>