[9] RFR: 8164591: sun/net/www/protocol/https/HttpsClient/ServerIdentityTest.java failed with SSLHandshakeException

[Artem Smotrakov]

Well, in this particular case it's not clear that it has the same issue 
with free port (at least to me). The exception occurred on client side, 
so it's not the case where we don't know where the handshake came from.

I can make this enhancement, but like I said I don't think it's going to 
help, so I would like to keep debug output on.

Artem


On 09/14/2016 06:39 PM, Xuelei Fan wrote:
> On 9/15/2016 9:23 AM, Artem Smotrakov wrote:
>> Hi Xuelei,
>>
>> For this one, I am not sure that it would help here since the test
>> failed after it already had started handshaking.
> It has the same issue as a free-port is used.  We don't actually know 
> the handshake is coming from the right client.
>
> Xuelei
>
>> I would prefer to have it as a separate enhancement.
>>
>> Artem
>>
>>
>> On 09/14/2016 06:19 PM, Xuelei Fan wrote:
>>> As you were already there, I would suggest to consider the
>>> SSLSocketSample.java template as the comment in JDK-8163924 review
>>> thread.
>>>
>>> Thanks,
>>> Xuelei
>>>
>>> On 9/15/2016 9:13 AM, Artem Smotrakov wrote:
>>>> Not urgent, but I would appreciate if someone can get a chance to look
>>>> at this.
>>>>
>>>> Artem
>>>>
>>>>
>>>> On 09/07/2016 03:17 PM, Artem Smotrakov wrote:
>>>>> Sending to net-dev at openjdk.java.net as well.
>>>>>
>>>>> Artem
>>>>>
>>>>>
>>>>> On 09/07/2016 12:28 PM, Artem Smotrakov wrote:
>>>>>> Hello,
>>>>>>
>>>>>> Please review the following patch for
>>>>>> sun/net/www/protocol/https/HttpsClient/ServerIdentityTest.java
>>>>>>
>>>>>> The test has been observed to fail a couple of times, but it's still
>>>>>> not clear why it failed because there is not much info in logs. The
>>>>>> patch updates the test to enable additional debug output, so that we
>>>>>> have more info if it fails next time.
>>>>>>
>>>>>> While looking at the test, I notices a couple of issues, but they
>>>>>> don't seem to cause these intermittent failures:
>>>>>> - The test sets system properties for JSSE in a loop, but JSSE
>>>>>> provider reads them only once while initialization. As a result, 
>>>>>> only
>>>>>> values which were set in the first iteration are actually used.
>>>>>> - The test doesn't close files and sockets sometimes.
>>>>>>
>>>>>> The patch also fixed the issues above, and there are a couple
>>>>>> cosmetic changes.
>>>>>>
>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8164591
>>>>>> Webrev: http://cr.openjdk.java.net/~asmotrak/8164591/webrev.00/
>>>>>>
>>>>>> Artem
>>>>>
>>>>
>>