RFR: 8294378: URLPermission constructor exception when using tr locale [v3]
Ryan Wallace
duke at openjdk.org
Fri Nov 11 11:48:51 UTC 2022
> 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
More information about the net-dev
mailing list