Skip to content

Conversation

@wenytang-ms
Copy link
Contributor

@wenytang-ms wenytang-ms commented Aug 7, 2025

while run npm run build-server with JDK 24, meeting such error
JAXP00010003: The length of entity "[xml]" is "100,001" that exceeds the "100,000" limit set by "jdk.xml.maxGeneralEntitySizeLimit".

JDK 24 contains changes to JAXP limits, see: https://bugs.openjdk.org/browse/JDK-8343022

update the build-sever script to build success with JDK 24+

@wenytang-ms wenytang-ms requested a review from Copilot August 7, 2025 03:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR addresses a build compatibility issue with JDK 24 by configuring JAXP XML entity size limits. The issue occurs when running npm run build-server with JDK 24, which has stricter default XML entity size limits that cause the build to fail.

  • Updates the Maven build command to include JVM options that disable XML entity size limits
  • Modifies the build script to set MAVEN_OPTS environment variable with the necessary JVM flags
  • Ensures compatibility with JDK 24+ while maintaining backward compatibility

wenytang-ms and others added 2 commits August 7, 2025 11:12
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@testforstephen
Copy link
Contributor

The CI failed, let's use a separate PR to fix the test failure.

@testforstephen testforstephen merged commit 0565240 into main Aug 7, 2025
4 of 6 checks passed
@wenytang-ms wenytang-ms deleted the wenyt/jdk24 branch August 7, 2025 05:21
@wenytang-ms
Copy link
Contributor Author

@testforstephen sure

@wenytang-ms wenytang-ms self-assigned this Aug 28, 2025
@wenytang-ms wenytang-ms added this to the 2025-08 milestone Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants