On Fri, 22 Nov 2024 23:17:17 GMT, Joe Wang <jo...@openjdk.org> wrote:

> Adjust JAXP Limits. Limits are adjusted as specified in the CSR.
> 
> Tests: 
>      Updated the config test with the new settings.
> 
>      Removed obsolete tests Bug6309988.java and 
> Bug4674384_MAX_OCCURS_Test.java, and files used by these tests. These tests 
> verified legacy properties. They came into the JDK from the JAXP and SJSXP 
> standalone libraries. Improvements were made before the integration into the 
> JDK via for example, SJSXP Issue 30. Though the issue tracker is no longer 
> accessible, the tests, e.g. IssueTracker30, are in the JAXP test suite.

Changes requested by mullan (Reviewer).

src/java.xml/share/conf/jaxp-strict.properties.template line 98:

> 96: jdk.xml.entityReplacementLimit=100000
> 97: #
> 98: # Limits the number of attributes an element can have. The default value 
> is 10000.

s/10000/200/

src/java.xml/share/conf/jaxp-strict.properties.template line 106:

> 104: jdk.xml.maxOccurLimit=5000
> 105: #
> 106: # Limits the maximum element depth. The default value is 0.

s/0/100/

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

PR Review: https://git.openjdk.org/jdk/pull/22337#pullrequestreview-2461864467
PR Review Comment: https://git.openjdk.org/jdk/pull/22337#discussion_r1858740236
PR Review Comment: https://git.openjdk.org/jdk/pull/22337#discussion_r1858736752

Reply via email to