adding rsockets support into JDK

Lu, Yingqi at
Thu Oct 4 22:28:50 UTC 2018

Hi Brian,

Please find version 14 of the patch available at

In this version, I use shared secrets mechanism to avoid changing two setXXX methods from “package private” to “protected” in SocketImpl. Please review and let me know if this is a reasonable change.

I also modified CSR accordingly at


From: Brian Burkhalter [mailto:brian.burkhalter at]
Sent: Wednesday, October 3, 2018 5:18 PM
To: Lu, Yingqi < at>
Cc: Aundhe, Shirish <shirish.aundhe at>; nio-dev at; Viswanathan, Sandhya <sandhya.viswanathan at>; Kaczmarek, Eric <eric.kaczmarek at>
Subject: Re: adding rsockets support into JDK

Hi Lucy,

On Oct 3, 2018, at 5:02 PM, Lu, Yingqi < at< at>> wrote:

Thank you for all the help here!

I also modified the CSR at by adding following lines to the end of it. I think we need to include all the API changes here?

Yes, all visible API changes need to be included.

In addition, we will change the access modifier on the following methods: ServerSocket(SocketImpl impl) from package private to protected setSocket(Socket soc) and setServerSocket(ServerSocket soc) from package private to protected

On these latter two items, I recall that Alan had an objection:

"On the API changes then I see that two protected setXXXX methods have been added to SocketImpl. I think we should try to avoid that, even if it means using the shared secrets mechanism.” [1]

Please review and let me know if there is anything missing.

Will do.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the nio-dev mailing list