> On 7 Oct 2016, at 21:33, Artem Smotrakov <artem.smotra...@oracle.com> wrote:
> Hello,
> Please review the patch below for 
> sun/net/www/protocol/https/HttpsClient/ProxyAuthTest.java test.
> The test has been observed to fail intermittently, but the failure is not 
> reproducible standalone. The patch updates the test to follow the approach 
> from SSLSocketSample.java
> http://hg.openjdk.java.net/jdk9/dev/jdk/file/1f044f413e6c/test/javax/net/ssl/templates/SSLSocketSample.java
> I merged OriginServer.java to ProxyAuthTest.java since only this test uses 
> that file. I also added a couple of new common methods to SSLTest.java. They 
> are not used by ProxyAuthTest.java, but can be useful in other tests.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8166530
> Webrev: http://cr.openjdk.java.net/~asmotrak/8166530/webrev.00/

From my point of view, this looks ok.


Reply via email to