RFR: 8280122: SupportedGroupsExtension should output "named groups" rather than "versions"

[Xue-Lei Andrew Fan]

On Tue, 18 Jan 2022 11:11:49 GMT, John Jiang <jjiang at 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