On Thu, 29 Jan 2026 02:17:02 GMT, Artur Barashev <[email protected]> wrote:

>> src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 979:
>> 
>>> 977:      * @since 27
>>> 978:      */
>>> 979:     public void setEnableCertificateCompression(
>> 
>> Need to specify the default value in the constructor.
>
> The convention in `SSLParameters` is not to specify the default values: the 
> rest of set methods in this file don't mention it. In our particular case the 
> default value is the value of `jdk.tls.enableCertificateCompression` system 
> property, and if such value is not set then it's `true`

The defaults are listed in the SSLParameters ctor: 
https://download.java.net/java/early_access/jdk26/docs/api/java.base/javax/net/ssl/SSLParameters.html#%3Cinit%3E()

So I think the default for this new param should also be listed if the 
application is creating the `SSLParameters` and not the provider implementation.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28682#discussion_r2741648461

Reply via email to