On Fri, 6 Dec 2024 18:23:48 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
>> Matthew Donovan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> fixed read call on the server side. > > test/jdk/javax/net/ssl/SSLSocket/Tls13PacketSize.java line 75: > >> 73: sslOS.write(appData); >> 74: sslOS.flush(); >> 75: sslIS.read(); > > The failure is caused by closing the socket before all the data sent by the > client is read. In order to read all the data, you need something like: > Suggestion: > > sslIS.read(appData, 1, appData.length-1); > > The failure is hard to reproduce. You might need to repeat the test a few > hundred times to see if it's gone. And it only affects Windows. I ran the test many hundreds of times and didn't see the error. I updated the code as you suggested and ran it 100s of times again without reproducing it. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22591#discussion_r1882528400
