[gnodet]

Backport of #28555

Cherry-pick of 44700f0 to camel-4.18.x.
Conflicts resolved automatically by backport bot agent.

[davsclaus]

CI fails to compile the test on camel-4.18.x: CometdAllowedTransportsTest imports org.apache.camel.test.junit6, which does not exist on this branch. It should use org.apache.camel.test.junit5 instead. Also, the "Allowed Transports and JSONP" doc section from #28555 is missing from this backport.

Claude Code on behalf of davsclaus

[oscerd]

Adapted the backport for camel-4.18.x:

• Migrated CometdAllowedTransportsTest to use camel-test-junit5 (org.apache.camel.test.junit5.CamelTestSupport).
• Removed try-with-resources on HttpClient (not AutoCloseable on Java 17).
• Added mirrored cometd-component.adoc under catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/docs/ with the "Allowed Transports and JSONP" section.
• Moved upgrade guide entry to the 4.18.5 to 4.18.6 section in camel-4x-upgrade-guide-4_18.adoc.

Antigravity on behalf of oscerd

[gnodet-bot]

Both concerns from @davsclaus's review are addressed in the second commit (CAMEL-26335: adapt the backport to camel-4.18.x):

1. junit5 import — CometdAllowedTransportsTest now correctly imports org.apache.camel.test.junit5.CamelTestSupport (not junit6) ✅
2. Missing doc section — The "Allowed Transports and JSONP" section is present in cometd-component.adoc, and the upgrade guide entry is correctly placed under "Upgrading from 4.18.5 to 4.18.6" in camel-4x-upgrade-guide-4_18.adoc ✅

Implementation review:

• CometdEndpoint.java: new allowedTransports parameter with sensible default ("websocket,long-polling" — excludes callback-polling/JSONP) and proper @UriParam annotation
• CometdComponent.java: correctly normalizes whitespace in transport names via Arrays.stream(...).map(String::trim).filter(!empty)
• Test coverage: 3 integration tests exercising default-rejects-JSONP, default-allows-long-polling, and explicit-JSONP-opt-in scenarios

LGTM — clean backport.

This review was generated by an AI agent, Hermès on behalf of @gnodet.