On Wed, 7 Aug 2024 18:01:19 GMT, Weijun Wang <[email protected]> wrote:
>> Francisco Ferrari Bihurriet has updated the pull request incrementally with
>> one additional commit since the last revision:
>>
>> Throw an IllegalArgumentException exception if
>> Security.setProperty("include", ...) is invoked.
>>
>> Co-authored-by: Martin Balao <[email protected]>
>> Co-authored-by: Francisco Ferrari Bihurriet <[email protected]>
>
> src/java.base/share/conf/security/java.security line 33:
>
>> 31: #
>> 32: # The special "include" property can be defined one or multiple times in
>> 33: # this file with a filesystem path value. The effect of each definition
>
> "this file". Is this precise? As described below, It can also appear in
> included files and the file "java.security.properties" points to.
>
> Maybe not an issue since no one will be confused. You decide whether to make
> any update.
I don't see "this file" as strictly necessary given the context so we can
remove it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16483#discussion_r1707731255
