Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AMQ-9388 - Exclude activemq-client-jakarta from camel-activemq #1117

Merged
merged 1 commit into from Nov 14, 2023

Conversation

cshannon
Copy link
Contributor

The current version of camel pulls in the activemq-client-jakarta jar which is not necessary as it no longer exists with ActiveMQ 6.0.0

Furthermore the version being pulled in is 5.18.2 which contains a critical CVE that was fixed in 5.18.3

The current version of camel pulls in the activemq-client-jakarta jar
which is not necessary as it no longer exists with ActiveMQ 6.0.0

Furthermore the version being pulled in is 5.18.2 which contains a
critical CVE that was fixed in 5.18.3
@mattrpav
Copy link
Contributor

mattrpav commented Nov 14, 2023

While activemq-client-jakarta is a relocation to 6.0.0, the way the camel-activemq pulls it in, Maven reactor isn't up-leveling it to 6.0.0 for us-- which would have made this a non-issue. Alas, we'll need to recut the release.

I see activemq BOM's in our future.

% mvn dependency:tree snippet

[INFO] +- org.apache.camel:camel-activemq:jar:4.1.0:compile
[INFO] |  \- org.apache.activemq:activemq-client-jakarta:jar:5.18.2:compile

Copy link
Contributor

@mattrpav mattrpav left a comment

Choose a reason for hiding this comment

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

LGTM, good catch.

@cshannon
Copy link
Contributor Author

I can merge this and then we need to revert the 6.0.0 preparation on the main branch and redo the release, I will let @jbonofre handle that.

@cshannon cshannon merged commit cbe7c6a into apache:main Nov 14, 2023
1 check was pending
@cshannon cshannon deleted the AMQ-9388 branch April 29, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants