RFR[9] JDK-8168064: sun/security/ssl/ServerHandshaker/AnonCipherWithWantClientAuth.java failed with "Received fatal alert: handshake_failure"

[Artem Smotrakov]

Hi John,

I think it is too late to set parameters for server socket in 
setServerApplication() because handshaking is already done at this point.

You can use setServerPeer() and setClientPeer() if you need to configure 
server socket. In this case, you need to follow SSLSocketSample.java to 
implement doServerSide().

Artem


On 10/24/2016 03:24 AM, John Jiang wrote:
> 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
>>>
>>
>>
>