On Fri, 27 Oct 2023 20:22:05 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> This belongs to the `if (inboundClientLen == null || inboundClientLen < 0)`
>> side. The else side stays the same.
>
> Precisely, it's
>
> if (inboundServerLen == null || inboundServerLen < 0) {
> maxInboundClientCertChainLen = globalPropSet ?
> maxCertificateChainLength : 8;;
> } else {
> maxInboundClientCertChainLen = inboundServerLen;
> }
>
> and nothing else is needed.
Updated as suggested. Changed code to:
if (inboundServerLen == null || inboundServerLen < 0) {
maxInboundServerCertChainLen = globalPropSet ?
maxCertificateChainLength : 10;
} else {
maxInboundServerCertChainLen = inboundServerLen;
}
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15163#discussion_r1376841191
