Code review request [JDK 8]: 7132248, sun/security/ssl/sun/net/www/protocol/https/HttpsURLConnection/CookieHttpsClientTest.java failing

Xuelei Fan Xuelei.Fan at Oracle.COM
Mon Jan 23 12:27:52 UTC 2012


Remove the serviceabilty-dev.

Thanks for the quick code review.

Xuelei

On 1/23/2012 8:25 PM, Alan Bateman wrote:
> On 23/01/2012 12:21, Xuelei Fan wrote:
>> Webrev:  http://cr.openjdk.java.net/~xuelei/7132248/webrev.00/
>>
>> In JDK 8, the regression tests of JSSE (HTTP/TLS) run in agentvm 
>> mode.  In agentvm mode, multiple threads may share the thread pool.  
>> SunJSSE implementation initialize the SSL/TLS context at the first 
>> time the context get loaded, and will not dynamically change the 
>> context anymore after the initialization. If a test case has 
>> initialized the context, another test case share the same thread will 
>> use the same context. New settings in the later will have no impact 
>> on the context.
>>
>> The cause of the bug is that some other test case updated the 
>> context, and CookieHttpsClientTest cannot setup the context as 
>> expected. Need to make the test case run in othervm mode.
>>
>> Thanks,
>> Xuelei
> This looks fine to me, thanks for jumping on this annoying test failure.
>
> -Alan.
>
> PS: cc'ing security-dev as I'm guessing you cc'ed serviceabilty-dev in 
> error.




More information about the security-dev mailing list