> 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 19 additional commits since the last revision: - 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 - 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 - ... and 9 more: https://git.openjdk.org/jdk/compare/66d8f3c8...207f3131 ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10903/files - new: https://git.openjdk.org/jdk/pull/10903/files/07aa9643..207f3131 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10903&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10903&range=00-01 Stats: 5685 lines in 299 files changed: 1292 ins; 3165 del; 1228 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
