RFR: 8286962: java/net/httpclient/ServerCloseTest.java failed once with ConnectException [v2]
Daniel Fuchs
dfuchs at openjdk.java.net
Wed Jun 15 11:40:37 UTC 2022
On Wed, 15 Jun 2022 10:49:23 GMT, Conor Cleary <ccleary at openjdk.org> wrote:
>> **Issue**
>> A failure in this test was observed whereby an unexpected connection from a client that was not created by the test caused the test to unsucessfully complete.
>>
>> **Solution**
>> When a connection is accepted by the ServerSocket, some verification of the Request URI and of the Request Path is conducted. If the client connection is found to be invalid or external to the test, the connection is closed. The ServerSocket will continue to accept new connections until the correct parameters are met. Once a valid connection is accepted, the test behaves exactly as it previously did.
>
> Conor Cleary has updated the pull request incrementally with one additional commit since the last revision:
>
> 8286962: Validate method & path before URI
LGTM. Please make sure the test is stable before integrating, and then drop me a note, and I will sponsor.
-------------
Marked as reviewed by dfuchs (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9155
More information about the net-dev
mailing list