Support keeping private maven registry for maven wrapper in only-script mode

[schuch]

When renovate suggests updates to the Maven wrapper, it does not retain hostnames of an internal Maven registry in the distributionUrl parameter.
The distributionUrl parameter is important if the wrapper is used in only-script mode, which is the new default.

The following wrapper modes¹ are available:

• only-script (new default since mvnw 3.3²)
• script
• bin (the old default i think)
• source

Example of a wrapper.properties file with only-script mode:

wrapperVersion=3.3.2
distributionType=only-script
distributionUrl=https://fqdn.of.internal.registry/internal-repo1-mirror-name/org/apache/maven/apache-maven/3.9.9/apache-maven-3.9.9-bin.zip

Currently renovate scans the wrapper.properties file for the Pattern /^(.*?)\/org\/apache\/maven\/wrapper\/³ to extract the existing registry's fqdn. But this pattern does only match, if the wrapper is used in the bin mode. (wrapperUrl property). Renovate then forcefully sets MVNW_REPOURL to repo.maven.apache.org which cannot be overridden by the renovate user.

I would be nice, if renovate would support retaining the registry hostname for the only-script mode, like it does for the bin mode⁴.

Reproduction

• Repo: https://github.com/schuch/renovate-maven-wrapper-only-script
• Example MR: https://github.com/schuch/renovate-maven-wrapper-only-script/pull/6/files#diff-1df564955cac198c2a317aceb01d86891160f1b185d9875b384b7d54819af358R19

You can see that Renovate is changing the distributionUrl. In the example, the public registry URL https://repo1.maven.org/maven2 (vs. https://repo.maven.apache.org/maven2) was used for the test, as the update command mvn wrapper:wrapper would otherwise not run successfully with a real private URL. But I think you get the idea, because the distributionUrl should have remained stable in this case as well.

Footnotes

1. Documentation about the wrapper modes: https://maven.apache.org/wrapper/#usage-with-or-without-binary-jar ↩

2. Maven Wrapper 3.3 Release Notes: https://github.com/apache/maven-wrapper/releases/tag/maven-wrapper-3.3.0 ↩

3. Relevant line of code: https://github.com/renovatebot/renovate/blob/87152d36e1f011fad38eb6fd73ed7f664f3ca0bd/lib/modules/manager/maven-wrapper/artifacts.ts#L188 ↩

4. Related Issue that lead to introduction of the feature to keep the internal registry hostname in bin mode: https://github.com/renovatebot/renovate/issues/20064 ↩

[github-actions[bot]]

Hi there,

Get your discussion fixed faster by creating a minimal reproduction. This means a repository dedicated to reproducing this issue with the minimal dependencies and config possible.

Before we start working on your issue we need to know exactly what's causing the current behavior. A minimal reproduction helps us with this. Discussions without reproductions are less likely to be converted to Issues.

Please follow these steps:

1. Read our guide on creating a minimal reproduction.
2. Go to our minimal reproduction template repository.
3. Select the Use this template button to create a new repository based on the template.
4. Work on the minimal reproduction in your own repository.
5. Fill out the information in your repository's README.md.
6. Add the link to your reproduction to the first post of your Discussion. If you are not the original author, you can post a new comment with the link.

If you need help with running renovate on your minimal reproduction repository, please refer to our Running Renovate guide.
Good luck,

The Renovate team

[schuch]

Added a link to a minimal reproduction to the description

[rarkins]

Converted to issue: #32213