On Tue, 18 Jan 2022 11:11:49 GMT, John Jiang <jji...@openjdk.org> wrote:

> MessageFormat messageFormat = new MessageFormat(
>     ""versions": '['{0}']'", Locale.ENGLISH);
> 
> In class SupportedGroupsExtension, the above "versions" should be "named 
> groups".

Nice catching!

src/java.base/share/classes/sun/security/ssl/SupportedGroupsExtension.java line 
115:

> 113:         public String toString() {
> 114:             MessageFormat messageFormat = new MessageFormat(
> 115:                 "\"named groups\": '['{0}']'", Locale.ENGLISH);

Per the TLS specification, it may be a more instinctive to use  "supported 
groups".  It is not a big concern, no additional approval is required to me if 
you want an update.

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

Marked as reviewed by xuelei (Reviewer).

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

Reply via email to