Skip to content

DPE-3220 Backport security fixes from Camel 4.14.x.#159

Open
andreasmattes wants to merge 5 commits into
tesb/camel-4.8.1.xfrom
bugfix/tesb/apache-camel-4.8.1.x/DPE-3220
Open

DPE-3220 Backport security fixes from Camel 4.14.x.#159
andreasmattes wants to merge 5 commits into
tesb/camel-4.8.1.xfrom
bugfix/tesb/apache-camel-4.8.1.x/DPE-3220

Conversation

@andreasmattes

Copy link
Copy Markdown

CVE-2026-40453 camel-coap backport from 4.14.6
CVE-2026-33454 camel-mail backport from 4.14.6

CVE-2026-40453 camel-coap backport from 4.14.6
CVE-2026-33454 camel-mail backport from 4.14.6
@andreasmattes andreasmattes requested a review from a team as a code owner June 2, 2026 17:00
@andreasmattes andreasmattes marked this pull request as draft June 2, 2026 17:00
CVE-2026-27172 camel-consul from 4.14.6
CVE-2026-40453 camel-google-pubsub from 4.14.6
CVE-2026-40453 camel-jms from 4.14.6
CVE-2026-40453 camel-sjms from 4.14.6
CVE-2026-40473 camel-mina from 4.14.6
CVE-2026-47323 camel-cxf-rest, camel-cxf-transport, camel-knative-http from 4.14.6
CVE-2026-40860 camel-jms, camel-sjms, camel-sjms2, camel-amqp, camel-activemq, camel-activemq6 from 4.14.7
CVE-2026-40858 camel-infinispan from 4.14.7
@andreasmattes andreasmattes force-pushed the bugfix/tesb/apache-camel-4.8.1.x/DPE-3220 branch 2 times, most recently from e9769b4 to 5b7c76f Compare June 8, 2026 14:45
@andreasmattes andreasmattes force-pushed the bugfix/tesb/apache-camel-4.8.1.x/DPE-3220 branch from 5b7c76f to 3f5e1fa Compare June 8, 2026 14:53
@andreasmattes andreasmattes marked this pull request as ready for review June 9, 2026 09:24
@essobedo

essobedo commented Jun 9, 2026

Copy link
Copy Markdown

Sounds weird, it looks like you backported a new feature introduced in 4.14 https://issues.apache.org/jira/browse/CAMEL-23222 along with its security fix. Is-it intended?

@essobedo

essobedo commented Jun 9, 2026

Copy link
Copy Markdown

Same remark for Google Pubsub https://issues.apache.org/jira/browse/CAMEL-22403. It is a new feature introduced in 4.14, so I don't get why we need to backport it to fix the CVE introduced by this feature.

@essobedo

essobedo commented Jun 9, 2026

Copy link
Copy Markdown

ClassicJmsHeaderFilterStrategy has been introduced in 4.9 cf https://issues.apache.org/jira/browse/CAMEL-21221. So for me it makes only sense for SjmsHeaderFilterStrategy and JmsHeaderFilterStrategy

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Why did you remove this test?

props.add("exceptionHandler");
props.add("exchangePattern");
props.add("headerFilterStrategy");
props.add("includeAllGoogleProperties");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm not sure it is a good idea to introduce includeAllGoogleProperties in a patch version

@essobedo essobedo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Congrats, huge work. I approve it as it is, but I still want to say that I'm not really fan of sometimes partial backports of features to get CVE fixes. It may be even more complex in the future to maintain this branch due to those partial backports.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants