RFR:8246330:Add TLS Tests for Legacy ECDSA curves

Rajan Halade RAJAN.HALADE at ORACLE.COM
Sat Jun 6 02:07:54 UTC 2020


One minor comment -

Can you please put try-catch around "(new DisabledCurve()).run();” instead of over for loop?

Otherwise test looks good.

Thanks,
Rajan

> On Jun 4, 2020, at 8:41 PM, shivangi.g.gupta at oracle.com wrote:
> 
> Corrected the links.
> 
> On 04/06/20 10:59 pm, shivangi.g.gupta at oracle.com <mailto:shivangi.g.gupta at oracle.com> wrote:
>> Hi, 
>> 
>> May I please find a sponsor for this patch? 
>> 
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8246330 <https://bugs.openjdk.java.net/browse/JDK-8246330> 
>> 
>> Description:Many of EC named curves has been disabled with security property "jdk.disabled.namedCurves". The purpose of this Test is to verify the behavior of any EC named curve from the disabled list. 
>> This Test will address a single name from the disabled list as sect283r1.        
>> 
>> 
>> Webrev: http://cr.openjdk.java.net/~sshivang/reviews/8246330/webrev.00/ <http://cr.openjdk.java.net/~sshivang/reviews/8246330/webrev.00/>
>> The patch has been tested on mach5, and the individual test was passed. 
>> 
>> Thanks 
>> 
>> Shivangi
>> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/security-dev/attachments/20200605/08474466/attachment.htm>


More information about the security-dev mailing list