On Tue, 26 Nov 2024 16:13:46 GMT, Sean Mullan <mul...@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.
>
> src/java.xml/share/classes/jdk/xml/internal/XMLSecurityManager.java line 101:
>
>> 99: */
>> 100: @SuppressWarnings("deprecation")
>> 101: public static enum Limit {
>
> Is it necessary to have both a default and secure value anymore? All the
> values are the same for both AFACT.
I'll start a discussion as related to the JEP, whether to keep FSP as the
approach has changed. For now, the values for enableExtensionFunctions are
still different. If there's a conclusion, I can make the change via that PR.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22337#discussion_r1858985140
