> Hi all, > > Please review this fix for Invalid characters in hostname exception. > This was due to the URLPermission.parseURI() method calling > String.toLowerCase() and returning a non ASCII characters in the case of the > tr Locale. The fix was to use the HostPortrange.toLowerCase() method further > down the stack which is Locale safe. > > Thanks, > Ryan.
Ryan Wallace has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 21 additional commits since the last revision: - 8294378: Remove toLowerCase for auth url string to mitigate non ascii characters generated in URLPermission and change url to lowercase in HostPortrange - Merge branch 'master' into 8294378 - Merge branch 'master' into 8294378 - Merge branch 'master' into 8294378 - 8294378: Remove toLowerCase for auth url string to mitigate non ascii characters generated in URLPermission and change url to lowercase in HostPortrange - Merge branch 'master' into 8294378 - Merge branch 'master' into 8294378 - Merge branch 'master' into 8294378 - Merge branch 'master' into 8294378 - Merge branch 'master' into 8294378 - ... and 11 more: https://git.openjdk.org/jdk/compare/cce07639...e992f1f3 ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10903/files - new: https://git.openjdk.org/jdk/pull/10903/files/207f3131..e992f1f3 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10903&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10903&range=01-02 Stats: 1496 lines in 90 files changed: 836 ins; 498 del; 162 mod Patch: https://git.openjdk.org/jdk/pull/10903.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10903/head:pull/10903 PR: https://git.openjdk.org/jdk/pull/10903
