On Fri, 27 Oct 2023 20:08:25 GMT, Weijun Wang <wei...@openjdk.org> wrote:

>> Hai-May Chao has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Change property names to clearly indicate client side and server side
>
> src/java.base/share/classes/sun/security/ssl/SSLConfiguration.java line 154:
> 
>> 152:             globalPropSet = true;
>> 153:         }
>> 154:         maxCertificateChainLength = certLen;
> 
> There is no need to set `certLen` or `maxCertificateChainLength` when 
> `globalPropSet` is false.

Removed them.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15163#discussion_r1376841208

Reply via email to