Re: [Internet]Re: [External] : Re: Recent SSLSocket close() @apiNote Changes.

2022-03-05 Thread xueleifan(XueleiFan)
> http://cr.openjdk.java.net/~wetmore/8282529/webrev.00 I was wondering if the mixing of half-close and duplex-close could work as well, by adjusting the implementation code. It could be easier for developers. But your spec update looks good to me, even if we allow the three-lines closure.

Re: [External] : Re: [Internet]Recent SSLSocket close() @apiNote Changes.

2022-03-04 Thread Bradford Wetmore
On 3/2/2022 11:46 PM, xueleifan(XueleiFan) wrote: I think you are right that this design is actually for TLSv1.3 half-close mode. For TLS 1.3, there is no duplex closure design. The close() implementation in JDK is actually a workaround for compatibility. Application can use either the

Re: [Internet]Recent SSLSocket close() @apiNote Changes.

2022-03-02 Thread xueleifan(XueleiFan)
Hi Brad, I think you are right that this design is actually for TLSv1.3 half-close mode. For TLS 1.3, there is no duplex closure design. The close() implementation in JDK is actually a workaround for compatibility. Application can use either the half-close mode socket.shutdownOutput();

Recent SSLSocket close() @apiNote Changes.

2022-03-02 Thread Bradford Wetmore
Hi Xuelei, I am working on some close code including the recent PR[1] for: 8282529: Fix API Note in javadoc for javax.net.ssl.SSLSocket and ran into a change I hadn't noticed before. * @apiNote * When the connection is no longer needed, the client and server * applications should