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