8251155: HostIdentifier fails to canonicalize hostnames starting with digits(Internet mail)

[jiefu(傅杰)]

Hi Serguei,


Thanks for your review and help.

Please see comments inline.

________________________________
From: serguei.spitsyn at oracle.com <serguei.spitsyn at oracle.com>
Sent: Wednesday, August 19, 2020 4:03 AM
To: jiefu(傅杰); serviceability-dev at openjdk.java.net; Claes Redestad
Subject: Re: 8251155: HostIdentifier fails to canonicalize hostnames starting with digits(Internet mail)


  83  * <ul>
  84  *   <li>{@code <null>} - transformed into "//localhost"</li>
  85  *   <li>localhost - transformed into "//localhost"</li>
  86  *   <li>hostname - transformed into "//hostname"</li>
  87  *   <li>hostname:port - transformed into "//hostname:port"</li>
  88  *   <li>proto:hostname - transformed into "proto://hostname"</li>
  89  *   <li>proto:hostname:port - transformed into
  90  *          "proto://hostname:port"</li>
  91  *   <li>proto://hostname:port</li>
  92  * </ul>

>> Is it worth to add an example to the list above?
Yes. It's really helpful for the review process. Thanks.


>> I wander if this fix needs a CSR.
I don't think so.
This is just a bug fix which doesn't add/remove/change any feature of the tools.
The original design has claimed to support hostname and hostname:port cases.
But it fails to do so when the hostname starts with digits.

It seems to be very common that the hostname will be started with digits in dockers.
So I think it's worth to fix this bug.


>> How did you check this fix does not introduce any regressions?
In fact, Claes had helped me to answer this question here: https://mail.openjdk.java.net/pipermail/serviceability-dev/2020-August/032691.html.
Also, I've tested this patch on Linux/x64 with tier1 ~ tier3 (no regression).


Thanks a lot.
Best regards,
Jie

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20200819/c6c84f18/attachment.htm>