Skip to content

CAMEL-23574: docs - sync camel-dns 4.18 / 4.14 upgrade-guide entries to main#23488

Open
oscerd wants to merge 1 commit into
apache:mainfrom
oscerd:CAMEL-23574-docsync-main
Open

CAMEL-23574: docs - sync camel-dns 4.18 / 4.14 upgrade-guide entries to main#23488
oscerd wants to merge 1 commit into
apache:mainfrom
oscerd:CAMEL-23574-docsync-main

Conversation

@oscerd
Copy link
Copy Markdown
Contributor

@oscerd oscerd commented May 24, 2026

Summary

Adds the camel-dns upgrade-guide entry for the header-constant rename
(CAMEL-23574) to the version-specific guides on main, matching the
backport-upgrade-guide policy (canonical guides live on main).

Original main PR: #23411
Backport PRs: #23459 (4.18.x, merged) · #23482 (4.14.x, open)

Test plan

  • Headings consistent across the three guides (all use the
    - potential breaking change suffix).
  • No code changes — docs-only PR.

Claude Code on behalf of Andrea Cosentino

…to main

The camel-dns header constant rename (CAMEL-23574, apache#23411) was backported to
camel-4.18.x (4.18.3) via apache#23459 and to camel-4.14.x (4.14.8) via apache#23482. The
version-specific upgrade-guide files on main are the canonical history, so
the corresponding entries are added here on main only.

Also normalises the existing 4_21 camel-dns heading to "- potential breaking
change" to match the suffix convention now used across the catalog-wide
header-rename family (CAMEL-23577) — e.g. camel-cxf, camel-jgroups,
camel-jira, camel-aws-bedrock.

Reported by Claude Code on behalf of Andrea Cosentino

Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions github-actions Bot added the docs label May 24, 2026
Copy link
Copy Markdown
Contributor

@gnodet gnodet left a comment

Choose a reason for hiding this comment

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

Docs-only PR syncing the camel-dns header-rename upgrade notes to the 4.18 and 4.14 guides on main, plus normalising the 4.21 heading suffix.

What I checked:

  • 4.14 & 4.18 entries: Identical content (correct — same rename applies to both lines). All 6 header renames documented accurately in a clear table format (dns.classCamelDnsClass, etc.).
  • Before/after code example: Helpful and correct — shows the literal-string migration path.
  • Cross-transport propagation note: Excellent addition. Correctly explains the security-relevant behavioural side-effect: Camel*-prefixed headers are filtered by HeaderFilterStrategy, so routes bridging external transports into dns: producers need explicit header mapping. This is a genuinely important migration caveat.
  • 4.21 heading normalisation: === camel-dns=== camel-dns - potential breaking change — consistent with the CAMEL-23577 sweep convention used across neighbouring entries.
  • Placement: 4.14 entry appends after the existing camel-jgroups section (correct position within the 4.14.8 upgrade section). 4.18 entry appends after the camel-cxf section.
  • No merge conflict risk with #23487: The two PRs touch different regions of camel-4x-upgrade-guide-4_14.adoc.

LGTM.

Claude Code on behalf of Guillaume Nodet

@github-actions
Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • docs

⚙️ View full build and test results

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants