On Fri, 10 May 2024 12:02:49 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8332020? >> >> `jwebserver` when it is launched prints a URL where the server is >> accessible. When launched using an IPv6 bind address, the printed URL >> doesn't enclose the IPv6 literal in `[` `]` thus rendering it in the form: >> >> URL http://0:0:0:0:0:0:0:1:8000/ >> >> This is an incorrect representation. As noted in RFC-2732 >> https://www.rfc-editor.org/rfc/rfc2732#section-2, the host literal is >> expected to be enclosed between `[` and `]`. >> >> The commit in this PR addresses that issues. A new jtreg test has been >> introduced to reproduce this issue and verify the fix. > > Jaikiran Pai has updated the pull request incrementally with one additional > commit since the last revision: > > Use -p 0 in the test > > Co-authored-by: Daniel Fuchs <67001856+df...@users.noreply.github.com> LGTM ------------- Marked as reviewed by dfuchs (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/19173#pullrequestreview-2050100758
