RFR: 8327989: java/net/httpclient/ManyRequest.java should not use "localhost" in URIs

Jaikiran Pai jpai at openjdk.org
Tue Mar 12 16:33:14 UTC 2024


On Tue, 12 Mar 2024 15:21:27 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

> Please find here a trivial test fix that removes usage of "localhost" from ManyRequest.java.
> Use of "localhost" in test URIs is problematic as it leaves the test open to random failure when running on machines that have custom mappings for "localhost". The fix is to use the IP literal of the loopback address instead.

The change looks good to me. The copyright year on the file would need an update before integrating.

-------------

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18239#pullrequestreview-1931580543


More information about the net-dev mailing list