Skip to content

[backport camel-4.18.x] CAMEL-23532: camel-vertx-websocket / camel-atmosphere-websocket / camel-iggy - use dedicated HeaderFilterStrategy aligned with sibling components#23313

Merged
oscerd merged 1 commit into
apache:camel-4.18.xfrom
oscerd:backport/23285-to-camel-4.18.x
May 19, 2026

Conversation

@oscerd
Copy link
Copy Markdown
Contributor

@oscerd oscerd commented May 19, 2026

Backport of #23285 to camel-4.18.x

Cherry-pick of #23285 (squash 1e776238202e) onto camel-4.18.x, for the 4.18.3 release line.

Original PR: #23285CAMEL-23532: camel-vertx-websocket / camel-atmosphere-websocket / camel-iggy - use dedicated HeaderFilterStrategy aligned with sibling components

Backport notes

  • iggy API divergence resolved: on camel-4.18.x, IggyFetchRecords uses message.userHeaders().ifPresent(...) with String keys / .value() (vs. main's Map / .toString() / .asString()). The HeaderFilterStrategy application was adapted to the 4.18.x shape (filtered per-entry loop inside the ifPresent block) rather than taking main's incompatible version.
  • Upgrade guide: the note was placed in camel-4x-upgrade-guide-4_18.adoc (the 4_21 guide does not exist on this branch). The matching entry on main's camel-4x-upgrade-guide-4_18.adoc is handled in a separate doc-sync PR.
  • Generated metadata (configurers, URI factories, component & catalog JSON, endpoint-DSL) regenerated against the 4.18.x baseline; unrelated stale-state regen drift reverted.
  • All three components compile on 4.18.x; VertxWebsocketHeaderFilterStrategyTest and IggyHeaderFilterStrategyTest pass (4/4 each).

Original description

Aligns camel-vertx-websocket, camel-atmosphere-websocket and camel-iggy with the HeaderFilterStrategy pattern already adopted across the component catalog (CAMEL-23222 camel-coap, CAMEL-23507 camel-cometd, CAMEL-23515 camel-nats). vertx-websocket and iggy gain a new *HeaderFilterStrategy plus a headerFilterStrategy endpoint option; atmosphere-websocket reuses the HeaderFilterStrategy it already inherits from the HTTP/servlet stack.


Claude Code on behalf of Andrea Cosentino

…el-iggy - use dedicated HeaderFilterStrategy aligned with sibling components

Apply a HeaderFilterStrategy on the inbound header mapping of the
vertx-websocket, atmosphere-websocket and iggy consumers, aligning them
with the pattern already used by camel-coap (CAMEL-23222), camel-cometd
(CAMEL-23507) and camel-nats (CAMEL-23515). vertx-websocket and iggy gain
a new VertxWebsocketHeaderFilterStrategy / IggyHeaderFilterStrategy plus a
headerFilterStrategy endpoint option; atmosphere-websocket reuses the
HeaderFilterStrategy it already inherits from the HTTP/servlet stack.
Includes unit tests and an upgrade-guide note.

Closes apache#23285

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
(cherry picked from commit 1e77623)
@oscerd oscerd merged commit 88c43e3 into apache:camel-4.18.x May 19, 2026
3 checks passed
oscerd added a commit that referenced this pull request May 19, 2026
…socket / camel-iggy 4.18 upgrade-guide entry to main (#23315)

The 4.18.x backport (#23313) adds the HeaderFilterStrategy upgrade-guide
note to camel-4x-upgrade-guide-4_18.adoc on the camel-4.18.x branch. Per
the project policy, the version-specific upgrade guides on main are the
canonical history across all releases, so the same entry must also be
added to camel-4x-upgrade-guide-4_18.adoc on main.

Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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