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

>> Hmm, but how does this work? The inbound properties override the global 
>> property if both are set.
>
> 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.

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

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

Reply via email to