gnodet opened a new pull request, #12685:
URL: https://github.com/apache/maven/pull/12685

   ## Summary
   
   - Add `maven-war-plugin` (min 3.4.0) to the `PLUGIN_UPGRADES` list in 
`PluginUpgradeStrategy`
   - Add `maven-ear-plugin` (min 3.4.0) to the `PLUGIN_UPGRADES` list in 
`PluginUpgradeStrategy`
   
   Both plugins use reflection on `java.util.Properties` internals (via 
XStream/plexus-archiver) in older versions, which is blocked by the JDK 17+ 
module system. Since Maven 4 requires JDK 17+, `mvnup` should upgrade these 
plugins to compatible versions.
   
   ### Affected versions
   
   | Plugin | Broken versions | Fix version | Root cause |
   |--------|----------------|-------------|------------|
   | `maven-war-plugin` | < 3.3.2 | 3.4.0 | XStream `PropertiesConverter` 
reflects on `Properties.defaults` field |
   | `maven-ear-plugin` | < 3.3.0 | 3.4.0 | `plexus-archiver` uses reflection 
blocked by JDK 17+ modules |
   
   ### Context
   
   Discovered during [Maven 4 compatibility 
testing](https://github.com/gnodet/maven4-testing) — projects like 
`mina-vysper` using `maven-war-plugin:2.1.1` crash with:
   ```
   java.lang.reflect.InaccessibleObjectException: Unable to make field 
protected volatile java.util.Properties java.util.Properties.defaults accessible
   ```
   
   Running `mvnup apply` didn't fix this because `maven-war-plugin` wasn't in 
the upgrade list.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to