RFR: 8216986 Remove unused code from SocksSocketImpl
Michael McMahon
michael.x.mcmahon at oracle.com
Wed Jan 23 14:30:52 UTC 2019
On 23/01/2019, 12:37, Alan Bateman wrote:
> On 23/01/2019 11:44, Michael McMahon wrote:
>> Hi
>>
>> Could I get the following webrev reviewed please?
>> It is just to remove dead code from SocksSocketImpl.
>> Most of the code was an (unused) attempt to implement
>> SOCKS for ServerSockets. getLocalPort() was potentially buggy
>> and should not override the super class implementation.
>> So, that was removed too. A couple of other orphaned methods
>> are removed also.
>>
>> http://cr.openjdk.java.net/~michaelm/8216986/webrev.1/
> This is a good cleanup. Seems wrong for getLocalPort to return the
> "remote" address of the SOCKS server so that fix looks correct. Does
> it need a test?
>
I can add a test for that.
> At some point I think we need to get this SocketImpl changed so that
> it delegates rather than extends PlainSocketImpl. I suspect that work
> will identify a few other oddities with SOCKS connections that may not
> be obvious now.
>
Yes, that's what I plan to do next.
Thanks,
Michael.
More information about the net-dev
mailing list