RFR [14] JDK-8233621 : Mismatch in jsse.enableMFLNExtension property name
Sean Mullan
sean.mullan at oracle.com
Fri Nov 8 20:58:49 UTC 2019
The fix looks good but I noticed that there is an existing test
(javax/net/ssl/TLSCommon/MFLNTest.java) that tests this feature using
the property name "jsse.enableMFLNExtension". That test should have
failed because of the misspelled property. I think you should look into
that to see why it didn't work properly - it may need some adjustment.
--Sean
On 11/8/19 12:22 PM, Xuelei Fan wrote:
> Hi,
>
> May I get the following update reviewed?
> http://cr.openjdk.java.net/~xuelei/8233621/webrev.00/
>
> Trivial update, no new regression test.
>
> The system property, "jsse.enableMFLNExtension", was introduced in JDK 9
> (See JSSE Reference Guides). However, the implementation code uses
> "jsse.enableMFLExtension" (without 'N') instead.
>
> With this update, temporarily, if either of the two properties set to
> true, the extension is switch on. Note that we may remove the
> "jsse.enableMFLExtension" property in the future. Please don't continue
> to use the misspelling property any longer.
>
> Thanks,
> Xuelei
More information about the security-dev
mailing list