On Fri, 4 Feb 2022 16:35:22 GMT, Sean Mullan <mul...@openjdk.org> wrote:

>> Xue-Lei Andrew Fan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   More update for the sec and impl
>
> src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 713:
> 
>> 711:      * in this list or may not use the recommended name for a certain
>> 712:      * signature scheme.
>> 713:      * <p>
> 
> Regarding my previous comment about providers not supporting these new 
> methods, I think something like the following might suffice:
> 
> 
> @apiNote Note that a provider may not have been updated to support this 
> method and in that case may return null instead of the default signature 
> schemes for pre-populated objects.
> @implNote The SunJSSE provider supports this method.

It looks like a typo. Replaced "pre-populated objects" with "connection 
populated objects", and added to the spec.

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

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

Reply via email to