Skip to content

build(deps)!: bump maven-core from 3.6.3 to 3.8.1#7612

Open
jeremylong wants to merge 1 commit intomainfrom
upgradeMaven
Open

build(deps)!: bump maven-core from 3.6.3 to 3.8.1#7612
jeremylong wants to merge 1 commit intomainfrom
upgradeMaven

Conversation

@jeremylong
Copy link
Collaborator

BREAKING CHANGE: dependency-check-maven now requires maven 3.8.1 or newer
Resolves #7566

BREAKING CHANGE: dependency-check-maven now requires maven 3.8.1 or newer
resolves #7566
@boring-cyborg boring-cyborg bot added ant changes to ant cli changes to the cli core changes to core documentation site documentation maven changes to the maven plugin utils changes to utils labels Apr 22, 2025
@jeremylong jeremylong requested a review from aikebah April 22, 2025 10:27
Copy link
Collaborator

@aikebah aikebah left a comment

Choose a reason for hiding this comment

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

LGTM; not required, but it saves us a whole lot of confusion and anyone interested in secure development pipelines should've upgraded to 3.8.1 or later anyhow.

Not sure how soon you'd like to release it as I can foresee my local attempts to get rid of the deprecated maven-artifact-transfer (https://github.com/apache/maven-artifact-transfer?tab=readme-ov-file#deprecation) as something that could likely trigger a new major (as it would be a good time to further cleanup/refactoring of the maven plugin amongst others addressing the plugin-dependencies-scope issue).

Hope to spend some serious time on that the week after ascension day.

@jeremylong
Copy link
Collaborator Author

@aikebah I'm fine holding off on publishing this so we can combine a few breaking changes. I don't see this PR as too high of a priority.

@johanblumenberg
Copy link

Any plans on merging this one?

@chadlwilson
Copy link
Collaborator

chadlwilson commented Feb 12, 2026

@johanblumenberg is there a particular issue you were trying to address?

More widely, Maven 3.8 is EOL now and will get no patches for security or otherwise, so that's food for thought in only supporting secure defaults, especially with supply chain risk increasing.

https://endoflife.date/apache-maven

Also 4.0 seems will be out soon, and I believe many plugins are being updated on their 3.8 or 3.9 compatible versions to be forward compatible so it'd be good to sort this to reduce the combinations we have to deal with.

I think we already have most of the infra already available to test using maven-invoker-plugin against multiple maven versions so I could take a look and see if there is any real difference between targeting a 3.8 or 3.9 minimum of the API.

I do agree with @aikebah that addressing the deprecated dependencies and classpath issues would probably be a good idea to do at the same time though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ant changes to ant cli changes to the cli core changes to core documentation site documentation maven changes to the maven plugin utils changes to utils

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pulling in old vulnerable version of maven-core

6 participants