On Tue, 18 Jan 2022 11:11:49 GMT, John Jiang <[email protected]> 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
