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