There is a similar one for HttpsURLConnection, as we had problems in the
past where the networking changes didn't update the SSL code.
jdk/sun/net/www/protocol/https/HttpsURLConnection/CheckMethods.java
Just a reminder to please include the JSSE reg tests when making
Socket/HttpsURLConnect
On 14/04/14 16:28, Alan Bateman wrote:
On 14/04/2014 16:30, Chris Hegarty wrote:
It looks like the change for JDK-8027221 missed an @build on the
dependent library type, Util.
This is just another sighting of the dreaded @library with -conc issue.
diff --git a/test/java/net/URLPermission/nstes
On 14/04/2014 16:30, Chris Hegarty wrote:
It looks like the change for JDK-8027221 missed an @build on the
dependent library type, Util.
This is just another sighting of the dreaded @library with -conc issue.
diff --git a/test/java/net/URLPermission/nstest/lookup.sh
b/test/java/net/URLPermiss
It looks like the change for JDK-8027221 missed an @build on the
dependent library type, Util.
This is just another sighting of the dreaded @library with -conc issue.
diff --git a/test/java/net/URLPermission/nstest/lookup.sh
b/test/java/net/URLPermission/nstest/lookup.sh
--- a/test/java/net/U
+1 . Thanks for updating this Xuelei. Nice test that caught this.
-Chris.
On 14/04/14 13:59, Sean Mullan wrote:
Looks fine to me. Can you add a relates to link to JDK-8036979 and an
appropriate noreg label?
--Sean
On 04/14/2014 06:04 AM, Xuelei Fan wrote:
Hi,
Please review the fix for JDK-8
On 4/14/2014 8:59 PM, Sean Mullan wrote:
> Looks fine to me. Can you add a relates to link to JDK-8036979 and an
> appropriate noreg label?
>
Made the update in JBS.
Thanks for the review.
Xuelei
> --Sean
>
> On 04/14/2014 06:04 AM, Xuelei Fan wrote:
>> Hi,
>>
>> Please review the fix for JDK-
Looks fine to me. Can you add a relates to link to JDK-8036979 and an
appropriate noreg label?
--Sean
On 04/14/2014 06:04 AM, Xuelei Fan wrote:
Hi,
Please review the fix for JDK-8040062:
http://cr.openjdk.java.net/~xuelei/8040062/webrev.00/
In JDK-8036979, there are news methods are ad
Hi,
Please review the fix for JDK-8040062:
http://cr.openjdk.java.net/~xuelei/8040062/webrev.00/
In JDK-8036979, there are news methods are added to java.net.Socket.
These methods need to be overridden in SSL socket implementation,
sun/security/ssl/BaseSSLSocketImpl.java.
No new regression
