RFR[12] JDK-8203687: javax/net/ssl/compatibility/Compatibility.java supports TLS 1.3

[Xuelei Fan]

Hi John,

Looks fine to me except a minor format comment.

Would you mind check the line length and limit to 80 characters each line?

For example, using "\"  join multiple lines together.
     // openssl req -x509 -new -key key.pem \
     //     -subj "/CN=RSA-2048-SHA256" -sha256 -out cer.pem

Thanks,
Xuelei

On 1/17/2019 12:26 AM, sha.jiang at oracle.com wrote:
> Hi,
> The patch adds TLS 1.3 cases for test 
> javax/net/ssl/compatibility/Compatibility.java.
> Beside this enhancement, it also changes the test on the following points:
> 1. Re-generate all certificates to use key size 2048 and SHA256 rather 
> than 1024 and SHA1. And new RSA signed EC key certificates are added.
> 2. Each JDK build should check if a cipher suite is supported by itself. 
> It's really unnecessary to maintain a predefined table for it.
> 3. Uses classes in javax/net/ssl/TLSCommon for defined protocols and 
> cipher suites. And also adds a new common class for key algorithm.
> 4. Correct some statements in README.
> 
> Webrev: http://cr.openjdk.java.net/~jjiang/8203687/webrev.00/
> Issue: https://bugs.openjdk.java.net/browse/JDK-8203687
> 
> Best regards,
> John Jiang
>