On Fri, 29 Jan 2021 00:50:04 GMT, Xue-Lei Andrew Fan wrote:
>> Clive Verghese has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments
>
> Please make sure all tier1 and tier2 tests passed on all supported platforms.
> E
> Redo for 8237578: JDK-8214339 (SSLSocketImpl wraps SocketException) appears
> to not be fully fixed
>
> This also fixes JDK-8259516: Alerts sent by peer may not be received
> correctly during TLS handshake
Clive Verghese has updated the pull request incrementally with one additional
commit s
On Mon, 1 Feb 2021 16:56:03 GMT, Daniel Fuchs wrote:
>> DatagramSocket has a DatagramSocket.setDatagramSocketImplFactory method that
>> allows to globally replace the default DatagramSocket/MulticastSocket
>> implementation provided by the JDK. This was provided as a way in early JDK
>> releas
On Mon, 1 Feb 2021 15:52:21 GMT, Daniel Fuchs wrote:
> > I think the title can be changed to something like "Update DatagramSocket
> > to add support for joining multicast groups" or something that makes it
> > clearer what the changes are about.
>
> Done. Let's see whether the bot deals with
On Sat, 30 Jan 2021 10:24:05 GMT, Alan Bateman wrote:
>> Daniel Fuchs has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fixup @bug in the tests
>
> src/java.base/share/classes/java/net/DatagramSocket.java line 223:
>
>> 221: *to
> DatagramSocket has a DatagramSocket.setDatagramSocketImplFactory method that
> allows to globally replace the default DatagramSocket/MulticastSocket
> implementation provided by the JDK. This was provided as a way in early JDK
> releases to replace the system wide implementation. It has been m
On Sat, 30 Jan 2021 08:15:01 GMT, Alan Bateman wrote:
> I think the title can be changed to something like "Update DatagramSocket to
> add support for joining multicast groups" or something that makes it clearer
> what the changes are about.
Done. Let's see whether the bot deals with the chang