On Thu, 7 Nov 2024 11:06:20 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
> This PR fixes an intermittent test failure on Windows. The test server used > in HttpURLConnectionExpect100Test now reads from the socket until the client > closes it. > > The current implementation closes the server socket after reading the request > headers, but without reading the request body. On Windows, if any data is > left in the socket receive buffer when the socket is closed, the connection > is reset. If the client doesn't read the response before the connection is > reset, the response is lost and the test fails. > > Verified that: > - the test still passes > - waiting for the client to close the socket does not take excessive amounts > of time (the test finishes in half a second on my machine) This pull request has now been integrated. Changeset: 72f67fb1 Author: Daniel Jeliński <djelin...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/72f67fb16a2470fc40c6ddc3700ba92f98a62096 Stats: 25 lines in 1 file changed: 7 ins; 15 del; 3 mod 8343411: Test fail on Windows due to "An established connection was aborted by the software in your host machine" Reviewed-by: dfuchs, jpai ------------- PR: https://git.openjdk.org/jdk/pull/21948
