Hi Serguei,
Thanks for your review and help. Please see comments inline. ________________________________ From: [email protected] <[email protected]> Sent: Wednesday, August 19, 2020 4:03 AM To: jiefu(傅杰); [email protected]; 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
