Ok, looks good.
--Sean
On 2/25/19 10:53 PM, Xuelei Fan wrote:
<Loop in net-dev>
It makes sense to me to leave the AbstractDelegateHttpsURLConnection
methods as public. We may need to re-org the code later, but it is out
of the scope of this update.
Here is the new webrev (only AbstractDelegateHttpsURLConnection updated):
http://cr.openjdk.java.net/~xuelei/8215430/webrev.01/
Thanks,
Xuelei
On 2/25/2019 1:55 PM, Sean Mullan wrote:
(I'd suggest cross-posting to net-dev since some classes in the
networking area are also changed).
- AbstractDelegateHttpsURLConnection
It might be less risky to leave the methods as public just in case
some code out there is using them (even though they are not supposed to).
The rest of this looks ok, pretty straightforward.
--Sean
On 2/21/19 10:45 PM, Xuelei Fan wrote:
Hi,
Could I get the following update reviewed:
http://cr.openjdk.java.net/~xuelei/8215430/webrev.00/
The internal package com.sun.net.ssl had been deprecated since JDK
1.4, and was not exported in the java.base module since JDK 9. It
should be safe to remove them in JDK 13.
For more details, please refer to CSR:
https://bugs.openjdk.java.net/browse/JDK-8218932
Thanks,
Xuelei
