Hi Artem,
Thanks for your review.
Would you like take a look at the updated webrev: http://cr.openjdk.java.net/~jjiang/8168064/webrev.01/ I also modified SSLTest.java a bit to expose SSLServerSocket instance to support the fixing.

Best regards,
John Jiang


On 2016/10/22 1:50, Artem Smotrakov wrote:
Hi John,

It may be better to use SSLTest.java to avoid duplicate code. The class basically contains parts of SSLSocketSample.java

http://hg.openjdk.java.net/jdk9/dev/jdk/file/0fb9ba19a63a/test/javax/net/ssl/templates/SSLTest.java

Here is an example

http://hg.openjdk.java.net/jdk9/dev/jdk/file/0fb9ba19a63a/test/sun/net/www/protocol/https/HttpsClient/ProxyAuthTest.java

Artem


On 10/20/2016 10:13 PM, John Jiang wrote:
Hi,
Please review this patch for fixing an intermittent issue on test sun/security/ssl/ServerHandshaker/AnonCipherWithWantClientAuth.java. The fix applies the pattern from test/javax/net/ssl/templates/SSLSocketSample.java

Webrev: http://cr.openjdk.java.net/~jjiang/8168064/webrev.00/
Issue: https://bugs.openjdk.java.net/browse/JDK-8168064

Best regards,
John Jiang




Reply via email to