RFR: 8303530: Add system property for custom JAXP configuration file [v2]

Alan Bateman alanb at openjdk.org
Mon Mar 13 11:37:23 UTC 2023


On Sat, 11 Mar 2023 05:34:39 GMT, Joe Wang <joehw at openjdk.org> wrote:

>> Add a system property, jdk.xml.config.file, to return the path to a custom JAXP configuration file. The current configuration file, jaxp.properties, that the JDK supports will become the default configuration file.
>> 
>> CSR: https://bugs.openjdk.org/browse/JDK-8303531
>> 
>> Tests: XML SQE and JCK tests passed.
>
> Joe Wang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   clean up tests; fix copy&paste error.

src/java.xml/share/classes/javax/xml/stream/XMLEventFactory.java line 139:

> 137:    *   Use the value of the property {@code factoryId} set in the
> 138:    * <a href="../../../module-summary.html#ConfigurationFile">configuration file</a>,
> 139:    * stax.properties or jaxp.properties by default. If the file exists and

Is it time to drop all references to stax.properties from the API docs?

src/java.xml/share/classes/module-info.java line 89:

> 87:  * the StAX factories for the <a href="#LookupMechanism">Factory Lookup Mechanism</a>
> 88:  * only. The files, if exist, are typically located in the {@code conf}
> 89:  * directory of the Java installation.

If jdk.xml.config.file is not set, should the reader assume it's default value is ${java.home}/conf/jaxp.properties? If so, they this is an implementation specific default, right?

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

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


More information about the core-libs-dev mailing list