RFR: 8280494: (D)TLS signature schemes [v13]

Xue-Lei Andrew Fan xuelei at openjdk.java.net
Mon Feb 7 22:24:11 UTC 2022


On Mon, 7 Feb 2022 19:59:32 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Xue-Lei Andrew Fan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   correct null tags
>
> src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 47:
> 
>> 45:  * <p>
>> 46:  * SSLParameters can be created via the constructors in this class.
>> 47:  * Objects can also be obtained using the {@code getSSLParameters()}
> 
> Since you introduce the terms "pre-populated" and "connection populated" in the new methods, I think it would be useful to describe them up front in the summary, ex:
> 
> `{@code SSLParameter} objects can be created via the constructors in this class, and can be described as pre-populated objects. {@code SSLParameter} objects can also be obtained using the ... <list all the methods>, and can be described as connection populated objects."

Nice!  It is something I want to add for quite a while, but never get a chance.  The suggested description is really simple and great!

Updated!

> src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 747:
> 
>> 745:      *
>> 746:      * @return an array of signature scheme {@code Strings} or {@code null} if
>> 747:      *         none have been set.  For non-null returns, this method willu
> 
> Typo, s/willu/will/

Thanks!

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

PR: https://git.openjdk.java.net/jdk/pull/7252



More information about the security-dev mailing list