RFR: 8220083: Use InetAddress.getLoopbackAddress() in place of 127.0.0.1 for some tests

[Arthur Eubanks]

Thanks for creating the umbrella bug.

Split long line.
Added the HTTPTestServer.java and UrgentDataTest.java changes in Chris's
suggestions. The other 2 involve URI/URLs, and there are a lot of those
changes, will group them in a later change.

PTAL: http://cr.openjdk.java.net/~aeubanks/8220083/webrev.02/


On Tue, Mar 12, 2019 at 5:00 AM Chris Hegarty <chris.hegarty at oracle.com>
wrote:

> Hi Arthur,
>
>
> On 11 Mar 2019, at 18:14, Arthur Eubanks <aeubanks at google.com> wrote:
>
> Updated copyright years (I asked around, it should be fine), updated
> commit message.
> http://cr.openjdk.java.net/~aeubanks/8220083/webrev.01
>
>
> This is a welcome improvement to the tests. Reviewed, with some minor
> comments and additions below.
>
> It will not be easy to get all the JDK networking tests passing
> successfully on IPv6-only environments ( ignoring the other combinations
> for now ). But this is a good improvement, and hopefully the start of
> a number of changes that add similar incremental improvements. To this
> end, I've created an umbrella task, 8220499, to track these test changes
> ( in one place ), and updated this issue, 8220083, to be a sub-task of
> it. Let's add more sub-tasks as needed.
>
> Specific webrev comment:
>  * Can you please split the long line in TunnelThroughProxy.java
>
> Suggestion for similar changes while here:
>  * http://cr.openjdk.java.net/~chegar/8220083.additional/index.html
>
> -Chris.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/net-dev/attachments/20190312/c8f221a9/attachment.html>