On Wed, 15 Sep 2021 12:00:07 GMT, Daniel Fuchs <[email protected]> wrote:
>> There is a bug for URLClassPath.findResources with JarIndex.
>> Currently, there was agreement on dropping the support from the
>> URLClassLoader implementation but it was suggested that it should be
>> disabled for a release or two before the code is removed.
>> A system property can be used to re-enable JarIndex support in URLClassPath.
>>
>> The PR includes:
>> Disable JarIndex support in URLClassPath by default.
>> Add system property jdk.net.URLClassPath.enableJarIndex to re-enable
>> JarIndex support.
>
> src/java.base/share/classes/jdk/internal/loader/URLClassPath.java line 115:
>
>> 113:
>> 114: p = props.getProperty("jdk.net.URLClassPath.enableJarIndex");
>> 115: ENABLE_JAR_INDEX = p != null ? p.equals("true") : false;
>
> Maybe this should use the same pattern than the other property above:
>
>
> ENABLE_JAR_INDEX = p != null ? p.equals("true") || p.isEmpty() : false;
Another question is where to document this property. We will obviously need a
CSR and releases notes. I also wonder if it should be promoted into a
networking property - which would make it possible to document it in the
`net.properties` file. @AlanBateman what do you think?
-------------
PR: https://git.openjdk.java.net/jdk/pull/5524
