Skip to content

fix(deps): replace jaxen 1.1.6 and exclude xmlParserAPIs 2.6.2 for Airlock - #2197

Merged
hj690 merged 1 commit into
mainfrom
fix-airlock-jaxen-xerces
Oct 22, 2026
Merged

hj690 merged 1 commit into
mainfrom
fix-airlock-jaxen-xerces

Conversation

@hj690

@hj690 hj690 commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

This PR removes the last two dependencies in google-http-java-client that have no approved license in Airlock (maven-3p-trusted): jaxen:jaxen:1.1.6 and xerces:xmlParserAPIs:2.6.2.

Changes

  • pom.xml: Override jaxen:jaxen to 1.2.0 (BSD-2-Clause, available in maven-3p-trusted) in findbugs-maven-plugin (3.0.5), which pulls jaxen:1.1.6 through com.google.code.findbugs:findbugs:3.0.1.
  • google-http-client-findbugs/pom.xml: Manage jaxen:jaxen at 1.2.0 in <dependencyManagement> for com.google.code.findbugs:findbugs:3.0.1.
  • google-http-client-android/pom.xml: Exclude xerces:xmlParserAPIs (2.6.2) from the provided-scope com.google.android:android:4.1.1.4 dependency, matching the existing exclusion in google-http-client-findbugs/pom.xml and google-http-client-assembly/pom.xml. Both the JDK and android-4.1.1.4.jar already provide the standard javax.xml.*, org.w3c.dom.*, and org.xml.sax.* APIs.

Verification

  • Ran mvn clean test package across all 15 modules and findbugs:check on dailymotion-simple-cmdline-sample in offline mode (-o) with jaxen:1.1.6 and xerces:xmlParserAPIs:2.6.2 removed from the local repository.
  • Confirmed that all 5,517 .class files and all OSGi META-INF/MANIFEST.MF headers across the 42 built JARs are byte-for-byte identical to main.

@hj690
hj690 requested a review from a team as a code owner October 22, 2026 17:26
@product-auto-label product-auto-label Bot added the size: s Pull request size is small. label Oct 22, 2026
@hj690
hj690 merged commit 46bff64 into main Oct 22, 2026
29 checks passed
@hj690
hj690 deleted the fix-airlock-jaxen-xerces branch October 22, 2026 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants