gnodet opened a new pull request, #12686:
URL: https://github.com/apache/maven/pull/12686
## Summary
- Pre-interpolates `PlexusConfiguration` values using the expression
evaluator in `DefaultMavenPluginManager` before passing them to the
`ComponentConfigurator`, ensuring `${...}` property references are fully
resolved before type converters (like `UriConverter`) process the values
- Fixes the case where properties set dynamically at runtime (e.g., by
Groovy/GMaven scripts via `project.properties.setProperty()`) are not available
during model interpolation and reach type converters unresolved, causing
`URISyntaxException` for URI-typed parameters
- Adds integration tests covering URI property interpolation with POM
properties, inherited parent properties, and CLI-passed properties
Regression found in CloudStack (gnodet/maven4-testing#34733) where a URI
parameter with `${cs.version}` set by a Groovy script at runtime caused
`URISyntaxException`.
## Test plan
- [x] Unit tests pass (580 tests in `impl/maven-core`)
- [x] New IT `MavenITmng8765UriPropertyInterpolationTest` passes:
- POM-defined properties in URI parameters (parent + child modules)
- CLI-passed properties (`-D`) in URI parameters
- [x] Existing plugin config ITs pass (MNG-3827, MNG-3864, MNG-0828, IT-0009)
- [ ] Full CI build
🤖 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]
