RFR: 8294378: URLPermission constructor exception when using tr locale [v2]

Jaikiran Pai jpai at openjdk.org
Fri Nov 11 03:47:43 UTC 2022


On Thu, 10 Nov 2022 15:06:57 GMT, Ryan Wallace <duke at openjdk.org> wrote:

>> 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/2725fee7...207f3131

test/jdk/java/net/URLPermission/URLPermissionTest.java line 2:

> 1: /*
> 2:  * Copyright (c) 2013, 2016, 2022, Oracle and/or its affiliates. All rights reserved.

Hello Ryan, this should be `2013, 2022,` - we only maintain 2 years the year this file was first introduced and the latest update year.

-------------

PR: https://git.openjdk.org/jdk/pull/10903


More information about the net-dev mailing list