RFR [13] JDK-8215430: Remove the internal package com.sun.net.ssl

Sean Mullan sean.mullan at oracle.com
Tue Feb 26 15:33:22 UTC 2019


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


More information about the security-dev mailing list